diff options
author | jan.nijtmans <nijtmans@users.sourceforge.net> | 2015-10-25 22:32:41 (GMT) |
---|---|---|
committer | jan.nijtmans <nijtmans@users.sourceforge.net> | 2015-10-25 22:32:41 (GMT) |
commit | bd597e95d63490d6c9668a27f18d8c4e7e975c10 (patch) | |
tree | 16b5653a385b8259c573d8c1ee9ca6391a10bcf1 /generic/tkOption.c | |
parent | 5ad0c4f5c60539e665a660b4b1b457a9e8d99169 (diff) | |
parent | 820c13e4bd18805cf590e0fa0e1627e7ecf92ec4 (diff) | |
download | tk-bd597e95d63490d6c9668a27f18d8c4e7e975c10.zip tk-bd597e95d63490d6c9668a27f18d8c4e7e975c10.tar.gz tk-bd597e95d63490d6c9668a27f18d8c4e7e975c10.tar.bz2 |
Fix [477949]: option readfile cannot use multibytes.
Implementation adopted from AndroWish, but added support for UTF-8 BOM and added test-case.
Diffstat (limited to 'generic/tkOption.c')
-rw-r--r-- | generic/tkOption.c | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/generic/tkOption.c b/generic/tkOption.c index 75dc3b9..d758b6f 100644 --- a/generic/tkOption.c +++ b/generic/tkOption.c @@ -1083,7 +1083,7 @@ ReadOptionFile( char *buffer; int result, bufferSize; Tcl_Channel chan; - Tcl_DString newName; + Tcl_DString newName, optString; /* * Prevent file system access in a safe interpreter. @@ -1135,7 +1135,16 @@ ReadOptionFile( } Tcl_Close(NULL, chan); buffer[bufferSize] = 0; - result = AddFromString(interp, tkwin, buffer, priority); + if ((bufferSize>2) && !memcmp(buffer, "\357\273\277", 3)) { + /* File starts with UTF-8 BOM */ + result = AddFromString(interp, tkwin, buffer+3, priority); + } else { + Tcl_DStringInit(&optString); + Tcl_ExternalToUtfDString(NULL, buffer, bufferSize, &optString); + result = AddFromString(interp, tkwin, Tcl_DStringValue(&optString), + priority); + Tcl_DStringFree(&optString); + } ckfree(buffer); return result; } |