From d9a6a00d31a183d425194924b689789e608dc7aa Mon Sep 17 00:00:00 2001 From: fvogel Date: Wed, 23 Mar 2016 16:55:30 +0000 Subject: Fixed [3512539fff] - .. is silently accepted in widget paths --- generic/tkWindow.c | 11 +++++++++-- tests/window.test | 4 +++- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/generic/tkWindow.c b/generic/tkWindow.c index b5cbbab..3797c23 100644 --- a/generic/tkWindow.c +++ b/generic/tkWindow.c @@ -1173,10 +1173,12 @@ Tk_CreateWindowFromPath( /* * Strip the parent's name out of pathName (it's everything up to the last - * dot). There are two tricky parts: (a) must copy the parent's name + * dot). There are three tricky parts: (a) must copy the parent's name * somewhere else to avoid modifying the pathName string (for large names, * space for the copy will have to be malloc'ed); (b) must special-case - * the situation where the parent is ".". + * the situation where the parent is "."; (c) the parent's name cannot be + * only 1 character long because it should include both a leading dot and + * at least one additional characters. */ p = strrchr(pathName, '.'); @@ -1195,6 +1197,11 @@ Tk_CreateWindowFromPath( if (numChars == 0) { *p = '.'; p[1] = '\0'; + } else if (numChars == 1) { + Tcl_SetObjResult(interp, Tcl_ObjPrintf( + "bad window path name \"%s\"", pathName)); + Tcl_SetErrorCode(interp, "TK", "VALUE", "WINDOW_PATH", NULL); + return NULL; } else { strncpy(p, pathName, (size_t) numChars); p[numChars] = '\0'; diff --git a/tests/window.test b/tests/window.test index fea695a..f02db3e 100644 --- a/tests/window.test +++ b/tests/window.test @@ -39,7 +39,9 @@ test window-1.1 {Tk_CreateWindowFromPath procedure, parent dead} -setup { while executing "button .t.b -text hello" (command bound to event)}} - +test window-1.2 {Tk_CreateWindowFromPath procedure, pathname starting with two dots} -body { + label ..mylabel +} -returnCodes error -result {bad window path name "..mylabel"} # Most of the tests below don't produce meaningful results; they # will simply dump core if there are bugs. -- cgit v0.12 From c5eb851f8387eb5fc52c3ab493c72f3e3d3f5f45 Mon Sep 17 00:00:00 2001 From: fvogel Date: Wed, 23 Mar 2016 17:00:44 +0000 Subject: Fixed typo in comment --- generic/tkWindow.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/generic/tkWindow.c b/generic/tkWindow.c index 3797c23..95e431c 100644 --- a/generic/tkWindow.c +++ b/generic/tkWindow.c @@ -1178,7 +1178,7 @@ Tk_CreateWindowFromPath( * space for the copy will have to be malloc'ed); (b) must special-case * the situation where the parent is "."; (c) the parent's name cannot be * only 1 character long because it should include both a leading dot and - * at least one additional characters. + * at least one additional character. */ p = strrchr(pathName, '.'); -- cgit v0.12