diff options
author | dgp <dgp@users.sourceforge.net> | 2012-10-24 19:21:41 (GMT) |
---|---|---|
committer | dgp <dgp@users.sourceforge.net> | 2012-10-24 19:21:41 (GMT) |
commit | dafa4d1504763f0809dfb3fd0ffa19f289a42888 (patch) | |
tree | 76a50d14d8f6d2c5a41b0fbd27b7ea2a17c2b640 /macosx/tkMacOSXWm.c | |
parent | 40af4f0e8c40ef3a1bc01189d45ddd921cc14550 (diff) | |
parent | 65a7afe8edb4a3261f46a2e244503dd85066e1fe (diff) | |
download | tk-dafa4d1504763f0809dfb3fd0ffa19f289a42888.zip tk-dafa4d1504763f0809dfb3fd0ffa19f289a42888.tar.gz tk-dafa4d1504763f0809dfb3fd0ffa19f289a42888.tar.bz2 |
3574893 Add overlooked toplevel ref count maintenance in the [wm manage|forget]
operations that could cause segfaults due to premature free of structs.
Diffstat (limited to 'macosx/tkMacOSXWm.c')
-rw-r--r-- | macosx/tkMacOSXWm.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/macosx/tkMacOSXWm.c b/macosx/tkMacOSXWm.c index 1bf6d2c..68cbb00 100644 --- a/macosx/tkMacOSXWm.c +++ b/macosx/tkMacOSXWm.c @@ -1659,7 +1659,9 @@ WmForgetCmd( TkFocusJoin(winPtr); Tk_UnmapWindow(frameWin); + macWin->toplevel->referenceCount--; macWin->toplevel = winPtr->parentPtr->privatePtr->toplevel; + macWin->toplevel->referenceCount++; macWin->flags &= ~TK_HOST_EXISTS; TkWmDeadWindow(winPtr); @@ -2448,7 +2450,9 @@ WmManageCmd( } wmPtr = winPtr->wmInfoPtr; winPtr->flags &= ~TK_MAPPED; + macWin->toplevel->referenceCount--; macWin->toplevel = macWin; + macWin->toplevel->referenceCount++; RemapWindows(winPtr, macWin); winPtr->flags |= (TK_TOP_HIERARCHY|TK_TOP_LEVEL|TK_HAS_WRAPPER|TK_WIN_MANAGED); @@ -6599,7 +6603,9 @@ RemapWindows( if (winPtr->window != None) { MacDrawable *macWin = (MacDrawable *) winPtr->window; + macWin->toplevel->referenceCount--; macWin->toplevel = parentWin->toplevel; + macWin->toplevel->referenceCount++; winPtr->flags &= ~TK_MAPPED; #ifdef TK_REBUILD_TOPLEVEL winPtr->flags |= TK_REBUILD_TOPLEVEL; |