NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit981dc2b

committed

Make ts_locale.c's character-type functions cope with UTF-16.

On Windows, in UTF8 database encoding, what char2wchar() produces isUTF16 not UTF32, ie, characters above U+FFFF will be represented bysurrogate pairs. t_isdigit() and siblings did not account for thisand failed to provide a large enough result buffer. That in turnled to bogus "invalid multibyte character for locale" errors, becausecontrary to what you might think from char2wchar()'s documentation,its Windows code path doesn't cope sanely with buffer overflow.The solution for t_isdigit() and siblings is pretty clear: providea 3-wchar_t result buffer not 2.char2wchar() also needs some work to provide more consistent, and moreaccurately documented, buffer overrun behavior. But that's a bigger joband it doesn't actually have any immediate payoff, so leave it for later.Per bug #15476 from Kenji Uno, who deserves credit for identifying thecause of the problem. Back-patch to all active branches.Discussion:https://postgr.es/m/15476-4314f480acf0f114@postgresql.org

1 parentdfa6081 commit981dc2bCopy full SHA for 981dc2b

File tree

1 file changed

+19

-8

lines changed

src/backend/tsearch
- ts_locale.c

1 file changed

+19

-8

lines changed

`‎src/backend/tsearch/ts_locale.c`

Lines changed: 19 additions & 8 deletions

Original file line number	Diff line number	Diff line change
`@@ -21,18 +21,29 @@`
`21`	`21`	`staticvoidtsearch_readline_callback(void*arg);`
`22`	`22`
`23`	`23`
	`24`	`+/*`
	`25`	`+ * The reason these functions use a 3-wchar_t output buffer, not 2 as you`
	`26`	`+ * might expect, is that on Windows "wchar_t" is 16 bits and what we'll be`
	`27`	`+ * getting from char2wchar() is UTF16 not UTF32. A single input character`
	`28`	`+ * may therefore produce a surrogate pair rather than just one wchar_t;`
	`29`	`+ * we also need room for a trailing null. When we do get a surrogate pair,`
	`30`	`+ * we pass just the first code to iswdigit() etc, so that these functions will`
	`31`	`+ * always return false for characters outside the Basic Multilingual Plane.`
	`32`	`+ */`
	`33`	`+#defineWC_BUF_LEN 3`
	`34`	`+`
`24`	`35`	`int`
`25`	`36`	`t_isdigit(constchar*ptr)`
`26`	`37`	`{`
`27`	`38`	`intclen=pg_mblen(ptr);`
`28`		`-wchar_tcharacter[2];`
	`39`	`+wchar_tcharacter[WC_BUF_LEN];`
`29`	`40`	`Oidcollation=DEFAULT_COLLATION_OID;/* TODO */`
`30`	`41`	`pg_locale_tmylocale=0;/* TODO */`
`31`	`42`
`32`	`43`	`if (clen==1\|\|lc_ctype_is_c(collation))`
`33`	`44`	`returnisdigit(TOUCHAR(ptr));`
`34`	`45`
`35`		`-char2wchar(character,2,ptr,clen,mylocale);`
	`46`	`+char2wchar(character,WC_BUF_LEN,ptr,clen,mylocale);`
`36`	`47`
`37`	`48`	`returniswdigit((wint_t)character[0]);`
`38`	`49`	`}`
`@@ -41,14 +52,14 @@ int`
`41`	`52`	`t_isspace(constchar*ptr)`
`42`	`53`	`{`
`43`	`54`	`intclen=pg_mblen(ptr);`
`44`		`-wchar_tcharacter[2];`
	`55`	`+wchar_tcharacter[WC_BUF_LEN];`
`45`	`56`	`Oidcollation=DEFAULT_COLLATION_OID;/* TODO */`
`46`	`57`	`pg_locale_tmylocale=0;/* TODO */`
`47`	`58`
`48`	`59`	`if (clen==1\|\|lc_ctype_is_c(collation))`
`49`	`60`	`returnisspace(TOUCHAR(ptr));`
`50`	`61`
`51`		`-char2wchar(character,2,ptr,clen,mylocale);`
	`62`	`+char2wchar(character,WC_BUF_LEN,ptr,clen,mylocale);`
`52`	`63`
`53`	`64`	`returniswspace((wint_t)character[0]);`
`54`	`65`	`}`
`@@ -57,14 +68,14 @@ int`
`57`	`68`	`t_isalpha(constchar*ptr)`
`58`	`69`	`{`
`59`	`70`	`intclen=pg_mblen(ptr);`
`60`		`-wchar_tcharacter[2];`
	`71`	`+wchar_tcharacter[WC_BUF_LEN];`
`61`	`72`	`Oidcollation=DEFAULT_COLLATION_OID;/* TODO */`
`62`	`73`	`pg_locale_tmylocale=0;/* TODO */`
`63`	`74`
`64`	`75`	`if (clen==1\|\|lc_ctype_is_c(collation))`
`65`	`76`	`returnisalpha(TOUCHAR(ptr));`
`66`	`77`
`67`		`-char2wchar(character,2,ptr,clen,mylocale);`
	`78`	`+char2wchar(character,WC_BUF_LEN,ptr,clen,mylocale);`
`68`	`79`
`69`	`80`	`returniswalpha((wint_t)character[0]);`
`70`	`81`	`}`
`@@ -73,14 +84,14 @@ int`
`73`	`84`	`t_isprint(constchar*ptr)`
`74`	`85`	`{`
`75`	`86`	`intclen=pg_mblen(ptr);`
`76`		`-wchar_tcharacter[2];`
	`87`	`+wchar_tcharacter[WC_BUF_LEN];`
`77`	`88`	`Oidcollation=DEFAULT_COLLATION_OID;/* TODO */`
`78`	`89`	`pg_locale_tmylocale=0;/* TODO */`
`79`	`90`
`80`	`91`	`if (clen==1\|\|lc_ctype_is_c(collation))`
`81`	`92`	`returnisprint(TOUCHAR(ptr));`
`82`	`93`
`83`		`-char2wchar(character,2,ptr,clen,mylocale);`
	`94`	`+char2wchar(character,WC_BUF_LEN,ptr,clen,mylocale);`
`84`	`95`
`85`	`96`	`returniswprint((wint_t)character[0]);`
`86`	`97`	`}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit981dc2b

File tree

1 file changed

1 file changed

`‎src/backend/tsearch/ts_locale.c`

0 commit comments