mirror of
https://github.com/gryf/wmaker.git
synced 2025-12-19 12:28:22 +01:00
Fix NULL pointer dereference
Some weird applications like Safari for Windows running on CXOffice or some applications running on Wine were causing wmaker to crash. So let's not rely on a window's `frame' property to be not NULL in src/wmspec.c. Original-post: http://lists.windowmaker.info/dev/msg00161.html See-also: http://lists.windowmaker.info/dev/msg00222.html
This commit is contained in:
committed by
Carlos R. Mafra
parent
d6c134f420
commit
a9aeefbe70
12
src/wmspec.c
12
src/wmspec.c
@@ -1215,11 +1215,13 @@ static void updateWindowType(WWindow * wwin)
|
||||
XFree(data);
|
||||
}
|
||||
|
||||
ChangeStackingLevel(wwin->frame->core, layer);
|
||||
wwin->frame->flags.need_texture_change = 1;
|
||||
wWindowConfigureBorders(wwin);
|
||||
wFrameWindowPaint(wwin->frame);
|
||||
wNETWMUpdateActions(wwin, False);
|
||||
if (wwin->frame != NULL) {
|
||||
ChangeStackingLevel(wwin->frame->core, layer);
|
||||
wwin->frame->flags.need_texture_change = 1;
|
||||
wWindowConfigureBorders(wwin);
|
||||
wFrameWindowPaint(wwin->frame);
|
||||
wNETWMUpdateActions(wwin, False);
|
||||
}
|
||||
}
|
||||
|
||||
Bool wNETWMCheckClientHints(WWindow * wwin, int *layer, int *workspace)
|
||||
|
||||
Reference in New Issue
Block a user