From 0aa2082226c7156d97ae4a66c508d1fd6ee421c4 Mon Sep 17 00:00:00 2001 From: patthoyts Date: Fri, 27 Feb 2009 23:04:39 +0000 Subject: [Bug 2645457] check for dead windows after calling Tk_MakeWindowExist to avoid a crash when mapping dead windows. --- ChangeLog | 5 +++++ generic/tkWindow.c | 9 ++++++++- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 43397ee..91099b0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2009-02-27 Pat Thoyts + + * generic/tkWindow.c: [Bug 2645457] check for dead windows after + calling Tk_MakeWindowExist to avoid a crash when mapping dead windows. + 2009-02-23 Pat Thoyts * win/rc/*.cur: [Patch 2513104] fix cursor hotspots (cjmcdonald) diff --git a/generic/tkWindow.c b/generic/tkWindow.c index 37b4a43..8c2d435 100644 --- a/generic/tkWindow.c +++ b/generic/tkWindow.c @@ -11,7 +11,7 @@ * See the file "license.terms" for information on usage and redistribution of * this file, and for a DISCLAIMER OF ALL WARRANTIES. * - * RCS: @(#) $Id: tkWindow.c,v 1.102 2008/12/28 13:08:39 dkf Exp $ + * RCS: @(#) $Id: tkWindow.c,v 1.103 2009/02/27 23:04:39 patthoyts Exp $ */ #include "tkInt.h" @@ -1644,6 +1644,13 @@ Tk_MapWindow( if (winPtr->window == None) { Tk_MakeWindowExist(tkwin); } + /* + * [Bug 2645457]: the previous call permits events to be processed and can + * lead to the destruction of the window under some conditions. + */ + if (winPtr->flags & TK_ALREADY_DEAD) { + return; + } if (winPtr->flags & TK_WIN_MANAGED) { /* * Lots of special processing has to be done for top-level windows. -- cgit v0.12