summaryrefslogtreecommitdiffstats
path: root/generic
diff options
context:
space:
mode:
authordgp <dgp@users.sourceforge.net>2004-09-09 17:12:12 (GMT)
committerdgp <dgp@users.sourceforge.net>2004-09-09 17:12:12 (GMT)
commit045fc77ff4e4fe4a8733883a90adc96d68fc6547 (patch)
tree134ce773473248553a80b38aa1b531c25018bd39 /generic
parentd6e816c7ffc49a84ca4030b72df0b629851ea9a1 (diff)
downloadtcl-045fc77ff4e4fe4a8733883a90adc96d68fc6547.zip
tcl-045fc77ff4e4fe4a8733883a90adc96d68fc6547.tar.gz
tcl-045fc77ff4e4fe4a8733883a90adc96d68fc6547.tar.bz2
Also corrected faulty prevention of [namespace import] cycles.
[Bug 1017299]
Diffstat (limited to 'generic')
-rw-r--r--generic/tclNamesp.c48
1 files changed, 27 insertions, 21 deletions
diff --git a/generic/tclNamesp.c b/generic/tclNamesp.c
index 5cc4ed9..27aff8d 100644
--- a/generic/tclNamesp.c
+++ b/generic/tclNamesp.c
@@ -19,7 +19,7 @@
* See the file "license.terms" for information on usage and redistribution
* of this file, and for a DISCLAIMER OF ALL WARRANTIES.
*
- * RCS: @(#) $Id: tclNamesp.c,v 1.31.2.2 2004/09/09 15:45:28 dgp Exp $
+ * RCS: @(#) $Id: tclNamesp.c,v 1.31.2.3 2004/09/09 17:12:13 dgp Exp $
*/
#include "tclInt.h"
@@ -1117,7 +1117,7 @@ Tcl_Import(interp, namespacePtr, pattern, allowOverwrite)
char *cmdName;
register Tcl_HashEntry *hPtr;
Tcl_HashSearch search;
- Command *cmdPtr, *realCmdPtr;
+ Command *cmdPtr;
ImportRef *refPtr;
Tcl_Command autoCmd, importedCmd;
ImportedCmdData *dataPtr;
@@ -1217,6 +1217,7 @@ Tcl_Import(interp, namespacePtr, pattern, allowOverwrite)
* pattern. Check whether it was exported. If it wasn't,
* we ignore it.
*/
+ Tcl_HashEntry *found;
wasExported = 0;
for (i = 0; i < importNsPtr->numExportPatterns; i++) {
@@ -1234,9 +1235,9 @@ Tcl_Import(interp, namespacePtr, pattern, allowOverwrite)
* Unless there is a name clash, create an imported command
* in the current namespace that refers to cmdPtr.
*/
-
- if ((Tcl_FindHashEntry(&nsPtr->cmdTable, cmdName) == NULL)
- || allowOverwrite) {
+
+ found = Tcl_FindHashEntry(&nsPtr->cmdTable, cmdName);
+ if ((found == NULL) || allowOverwrite) {
/*
* Create the imported command and its client data.
* To create the new command in the current namespace,
@@ -1254,25 +1255,30 @@ Tcl_Import(interp, namespacePtr, pattern, allowOverwrite)
/*
* Check whether creating the new imported command in the
- * current namespace would create a cycle of imported->real
- * command references that also would destroy an existing
- * "real" command already in the current namespace.
+ * current namespace would create a cycle of imported
+ * command references.
*/
cmdPtr = (Command *) Tcl_GetHashValue(hPtr);
- if (cmdPtr->deleteProc == DeleteImportedCmd) {
- realCmdPtr = (Command *) TclGetOriginalCommand(
- (Tcl_Command) cmdPtr);
- if ((realCmdPtr != NULL)
- && (realCmdPtr->nsPtr == currNsPtr)
- && (Tcl_FindHashEntry(&currNsPtr->cmdTable,
- cmdName) != NULL)) {
- Tcl_AppendStringsToObj(Tcl_GetObjResult(interp),
- "import pattern \"", pattern,
- "\" would create a loop containing command \"",
- Tcl_DStringValue(&ds), "\"", (char *) NULL);
- Tcl_DStringFree(&ds);
- return TCL_ERROR;
+ if ((found != NULL)
+ && cmdPtr->deleteProc == DeleteImportedCmd) {
+
+ Command *overwrite = (Command *) Tcl_GetHashValue(found);
+ Command *link = cmdPtr;
+ while (link->deleteProc == DeleteImportedCmd) {
+ ImportedCmdData *dataPtr;
+
+ dataPtr = (ImportedCmdData *) link->objClientData;
+ link = dataPtr->realCmdPtr;
+ if (overwrite == link) {
+ Tcl_AppendStringsToObj(Tcl_GetObjResult(interp),
+ "import pattern \"", pattern,
+ "\" would create a loop containing ",
+ "command \"", Tcl_DStringValue(&ds),
+ "\"", (char *) NULL);
+ Tcl_DStringFree(&ds);
+ return TCL_ERROR;
+ }
}
}