Project

General

Profile

Actions
Copy link

Bug #12323

closed
AA MV

WMenu::addSeparator() causing segfault

Bug #12323: WMenu::addSeparator() causing segfault

Added by Andrey Alekseev over 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
High
Assignee:
Matthias Van Ceulebroeck
Target version:
4.11.0
Start date:
01/12/2024
Due date:
% Done:

100%

Estimated time:
0:00 h

Description

Debugger says that segfault happens in WMenuItem::renderSelected()
I do not have time to debug and recompile WT at the moment, but as I can see here, possible nullptr, returned by WMenuItem::anchor() is not checked.
Actually, disabling Bootstrap 5 theme helped.

MV Updated by Matthias Van Ceulebroeck over 2 years ago Actions
Copy link
 #1

  • Status changed from New to InProgress
  • Assignee set to Matthias Van Ceulebroeck
  • Target version set to 4.11.0

Indeed, this is a forgotten check!

MV Updated by Matthias Van Ceulebroeck over 2 years ago Actions
Copy link
 #2

  • Status changed from InProgress to Review
  • Assignee deleted (Matthias Van Ceulebroeck)

KD Updated by Korneel Dumon over 2 years ago Actions
Copy link
 #3

  • Status changed from Review to InProgress
  • Assignee set to Matthias Van Ceulebroeck

MV Updated by Matthias Van Ceulebroeck over 2 years ago Actions
Copy link
 #4

  • Status changed from InProgress to Review
  • Assignee deleted (Matthias Van Ceulebroeck)

MV Updated by Matthias Van Ceulebroeck over 2 years ago Actions
Copy link
 #5

  • Assignee set to Korneel Dumon

KD Updated by Korneel Dumon over 2 years ago Actions
Copy link
 #6

  • Status changed from Review to InProgress
  • Assignee changed from Korneel Dumon to Matthias Van Ceulebroeck

CM Updated by Christian Meyer about 2 years ago Actions
Copy link
 #8

Didn't see this and opened #12900 with the same content

MV Updated by Matthias Van Ceulebroeck almost 2 years ago Actions
Copy link
 #9

  • Status changed from InProgress to Implemented @Emweb
  • % Done changed from 0 to 100

MV Updated by Matthias Van Ceulebroeck almost 2 years ago Actions
Copy link
 #10

  • Status changed from Implemented @Emweb to Closed
Actions
Copy link

Also available in: PDF Atom