summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjan.nijtmans <nijtmans@users.sourceforge.net>2017-01-11 11:37:21 (GMT)
committerjan.nijtmans <nijtmans@users.sourceforge.net>2017-01-11 11:37:21 (GMT)
commitec5e2f38d3eb0132822396f9b105d4cc5c78e86b (patch)
tree68dc8a4d9a6b01fe2552e0189e05f9455cdf9887
parent5947e3261525f012e16d814aca14fa42a498a5ce (diff)
parent2a52c1f6b17be29292bd301a7ee90157e856e6f7 (diff)
downloadtk-ec5e2f38d3eb0132822396f9b105d4cc5c78e86b.zip
tk-ec5e2f38d3eb0132822396f9b105d4cc5c78e86b.tar.gz
tk-ec5e2f38d3eb0132822396f9b105d4cc5c78e86b.tar.bz2
Fix [d4fb4e80d220e46e588f310291fd7a4205e8cd67|d4fb4e80d2]: Image photo commands read/write/put do not accept "-" as first letter in file names resp. image data
-rw-r--r--generic/tkImgPhoto.c6
-rw-r--r--tests/imgPhoto.test10
2 files changed, 15 insertions, 1 deletions
diff --git a/generic/tkImgPhoto.c b/generic/tkImgPhoto.c
index 4ee9f69..1bd0142 100644
--- a/generic/tkImgPhoto.c
+++ b/generic/tkImgPhoto.c
@@ -1540,7 +1540,11 @@ ParseSubcommandOptions(
*/
if (!(allowedOptions & bit)) {
- goto unknownOrAmbiguousOption;
+ if (optPtr->name != NULL) {
+ goto unknownOrAmbiguousOption;
+ }
+ optPtr->name = objv[index];
+ continue;
}
/*
diff --git a/tests/imgPhoto.test b/tests/imgPhoto.test
index db23fea..e93dab4 100644
--- a/tests/imgPhoto.test
+++ b/tests/imgPhoto.test
@@ -804,6 +804,16 @@ test imgPhoto-4.74 {ImgPhotoCmd procedure: put option error handling} -setup {
} -cleanup {
image delete photo1
} -returnCodes 1 -result {wrong # args: should be "photo1 put data ?-option value ...?"}
+test imgPhoto-4.75 {<photo> read command: filename starting with '-'} -constraints {
+ hasTeapotPhoto
+} -body {
+ file copy -force $teapotPhotoFile -teapotPhotoFile
+ image create photo photo1
+ photo1 read -teapotPhotoFile
+} -cleanup {
+ image delete photo1
+ file delete ./-teapotPhotoFile
+} -result {}
test imgPhoto-5.1 {ImgPhotoGet/Free procedures, shared instances} -constraints {
hasTeapotPhoto