# [PATCH] gitk: avoid crash if closed while reading references

2 messages from 2009-04-15 to 2009-04-15. Participants: Pat Thoyts, Johannes Schindelin.
Thread: https://gitlist.dev/t/18884

## Pat Thoyts, 2009-04-15 16:14

Subject: [PATCH] gitk: avoid crash if closed while reading references
Message-ID: <87prfdnbck.fsf@users.sourceforge.net>
URL: https://gitlist.dev/e/87prfdnbck.fsf%40users.sourceforge.net

```

  As recorded in msysGit issue 125 if the user closes gitk while it
  reports itself as still reading references then Tk will crash in the
  geometry management code. This has been fixed for Tk 8.5.7 and above.
  This patch avoids the problem by flushing outstanding geometry events
  before calling the readrefs procedure.

  See also http://code.google.com/p/msysgit/issues/detail?id=125

Signed-off-by: Pat Thoyts <patthoyts@users.sourceforge.net>
---
 gitk |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/gitk b/gitk
index 656c2c1..a346bd5 100755
--- a/gitk
+++ b/gitk
@@ -10952,6 +10952,7 @@ catch {
 # wait for the window to become visible
 tkwait visibility .
 wm title . "[file tail $argv0]: [file tail [pwd]]"
+update
 readrefs
 
 if {$cmdline_files ne {} || $revtreeargs ne {} || $revtreeargscmd ne {}} {
-- 
1.6.2.2.1669.g7eaf8

```

## Johannes Schindelin, 2009-04-15 16:46

Subject: Re: [PATCH] gitk: avoid crash if closed while reading references
Message-ID: <alpine.DEB.1.00.0904151846060.6814@intel-tinevez-2-302>
URL: https://gitlist.dev/e/alpine.DEB.1.00.0904151846060.6814%40intel-tinevez-2-302
In-Reply-To: <87prfdnbck.fsf@users.sourceforge.net>

```
Hi,

On Wed, 15 Apr 2009, Pat Thoyts wrote:

> 
>   As recorded in msysGit issue 125 if the user closes gitk while it
>   reports itself as still reading references then Tk will crash in the
>   geometry management code. This has been fixed for Tk 8.5.7 and above.
>   This patch avoids the problem by flushing outstanding geometry events
>   before calling the readrefs procedure.
> 
>   See also http://code.google.com/p/msysgit/issues/detail?id=125
> 
> Signed-off-by: Pat Thoyts <patthoyts@users.sourceforge.net>

This is commit 3d3ecb65e34a62b83648516150ec8a99e1b3ace7 in 4msysgit's 
'devel' branch now, and will be part of the next msysGit release.

Thanks,
Dscho

```