diff options
| author | dkf <donal.k.fellows@manchester.ac.uk> | 2005-06-14 13:45:57 (GMT) |
|---|---|---|
| committer | dkf <donal.k.fellows@manchester.ac.uk> | 2005-06-14 13:45:57 (GMT) |
| commit | 9480f190e5c0b49511894e60db3c34b9a7a81003 (patch) | |
| tree | 8753974f9a48a1a793cb9bd53171a42b17f86238 /generic/tclBasic.c | |
| parent | cdd600c79b33458ca7ca7861563f1c6756e69ff6 (diff) | |
| download | tcl-9480f190e5c0b49511894e60db3c34b9a7a81003.zip tcl-9480f190e5c0b49511894e60db3c34b9a7a81003.tar.gz tcl-9480f190e5c0b49511894e60db3c34b9a7a81003.tar.bz2 | |
Fix [Bug 1220058] and quash a bizarre case which generated a bogus error msg.
Thanks to Will Duquette for helping to track this one down.
Diffstat (limited to 'generic/tclBasic.c')
| -rw-r--r-- | generic/tclBasic.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/generic/tclBasic.c b/generic/tclBasic.c index 6935782..42f5465 100644 --- a/generic/tclBasic.c +++ b/generic/tclBasic.c @@ -13,7 +13,7 @@ * See the file "license.terms" for information on usage and redistribution * of this file, and for a DISCLAIMER OF ALL WARRANTIES. * - * RCS: @(#) $Id: tclBasic.c,v 1.160 2005/06/06 23:45:43 dkf Exp $ + * RCS: @(#) $Id: tclBasic.c,v 1.161 2005/06/14 13:46:02 dkf Exp $ */ #include "tclInt.h" @@ -2504,11 +2504,16 @@ Tcl_DeleteCommandFromToken(interp, cmd) /* * Another deletion is already in progress. Remove the hash * table entry now, but don't invoke a callback or free the - * command structure. + * command structure. Take care to only remove the hash entry + * if it has not already been removed; otherwise if we manage + * to hit this function three times, everything goes up in + * smoke. [Bug 1220058] */ - Tcl_DeleteHashEntry(cmdPtr->hPtr); - cmdPtr->hPtr = NULL; + if (cmdPtr->hPtr != NULL) { + Tcl_DeleteHashEntry(cmdPtr->hPtr); + cmdPtr->hPtr = NULL; + } return 0; } |