diff options
| author | Miguel Sofer <miguel.sofer@gmail.com> | 2003-02-18 02:22:32 (GMT) |
|---|---|---|
| committer | Miguel Sofer <miguel.sofer@gmail.com> | 2003-02-18 02:22:32 (GMT) |
| commit | d86b1af8bec78fdbcc8bf65bc205fd287e19fd5d (patch) | |
| tree | 31a14f5685b47234e304a11f204cce379f1e9003 | |
| parent | 67083d99f9986ffc35b561cd80ad68a6a21f19b7 (diff) | |
| download | tcl-d86b1af8bec78fdbcc8bf65bc205fd287e19fd5d.zip tcl-d86b1af8bec78fdbcc8bf65bc205fd287e19fd5d.tar.gz tcl-d86b1af8bec78fdbcc8bf65bc205fd287e19fd5d.tar.bz2 | |
* generic/tclBasic.c (TclRenameCommand): 'oldFullName' object was
not being freed on all function exits, causing a memory leak
[Bug 684756]
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | generic/tclBasic.c | 20 |
2 files changed, 19 insertions, 7 deletions
@@ -1,3 +1,9 @@ +2003-02-17 Miguel Sofer <msofer@users.sf.net> + + * generic/tclBasic.c (TclRenameCommand): 'oldFullName' object was + not being freed on all function exits, causing a memory leak + [Bug 684756] + 2003-02-17 Mo DeJong <mdejong@users.sourceforge.net> * generic/tclIO.c (Tcl_GetsObj): Minor change diff --git a/generic/tclBasic.c b/generic/tclBasic.c index 45f1422..cd42855 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.73 2003/02/16 01:36:32 msofer Exp $ + * RCS: @(#) $Id: tclBasic.c,v 1.74 2003/02/18 02:22:33 msofer Exp $ */ #include "tclInt.h" @@ -1947,7 +1947,8 @@ TclRenameCommand(interp, oldName, newName) if ((newName == NULL) || (*newName == '\0')) { Tcl_DeleteCommandFromToken(interp, cmd); - return TCL_OK; + result = TCL_OK; + goto done; } /* @@ -1964,13 +1965,16 @@ TclRenameCommand(interp, oldName, newName) Tcl_AppendStringsToObj(Tcl_GetObjResult(interp), "can't rename to \"", newName, "\": bad command name", (char *) NULL); - return TCL_ERROR; + Tcl_DecrRefCount( oldFullName ); + result = TCL_ERROR; + goto done; } if (Tcl_FindHashEntry(&newNsPtr->cmdTable, newTail) != NULL) { Tcl_AppendStringsToObj(Tcl_GetObjResult(interp), "can't rename to \"", newName, "\": command already exists", (char *) NULL); - return TCL_ERROR; + result = TCL_ERROR; + goto done; } /* @@ -2002,7 +2006,7 @@ TclRenameCommand(interp, oldName, newName) Tcl_DeleteHashEntry(cmdPtr->hPtr); cmdPtr->hPtr = oldHPtr; cmdPtr->nsPtr = cmdNsPtr; - return result; + goto done; } /* @@ -2028,7 +2032,6 @@ TclRenameCommand(interp, oldName, newName) Tcl_GetString( oldFullName ), Tcl_DStringValue( &newFullName ), TCL_TRACE_RENAME); - Tcl_DecrRefCount( oldFullName ); Tcl_DStringFree( &newFullName ); /* @@ -2056,8 +2059,11 @@ TclRenameCommand(interp, oldName, newName) * been deleted by invocation of rename traces. */ TclCleanupCommand(cmdPtr); + result = TCL_OK; - return TCL_OK; + done: + TclDecrRefCount( oldFullName ); + return result; } /* |