smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2026-01-08 22:55:44 +01:00
Code Issues Projects Releases Wiki Activity

NULL pointer dereference?

Browse source
This commit is contained in:
Robin Gareus 2014-11-17 03:58:05 +01:00
parent ad739cd018
commit 55ce10f498
1 changed files with 9 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

12
gtk2_ardour/editor.cc
View file

@ -4288,7 +4288,9 @@ Editor::undo_visual_state ()
redo_visual_stack.push_back (current_visual_state (vs ? vs->gui_state != 0 : false));
use_visual_state (*vs);
if (vs) {
use_visual_state (*vs);
}
}
void
@ -4301,9 +4303,13 @@ Editor::redo_visual_state ()
VisualState* vs = redo_visual_stack.back();
redo_visual_stack.pop_back();
undo_visual_stack.push_back (current_visual_state (vs ? vs->gui_state != 0 : false));
// can 'vs' really be 0? Is there a place that puts NULL pointers onto the stack?
// why do we check here?
undo_visual_stack.push_back (current_visual_state (vs ? (vs->gui_state != 0) : false));
use_visual_state (*vs);
if (vs) {
use_visual_state (*vs);
}
}
void