diff options
author | Alexander Barkov <bar@mariadb.com> | 2020-05-07 19:20:17 +0400 |
---|---|---|
committer | Alexander Barkov <bar@mariadb.com> | 2020-05-09 16:01:30 +0400 |
commit | cfe5ee90c8e4b9dfa98a41fcd299197a59261be7 (patch) | |
tree | 35fdaabac55d4b36d228bc9600112e986850b162 /sql/sql_error.cc | |
parent | c675886dcdecd29571bd08605a409325ee81004c (diff) | |
download | mariadb-git-cfe5ee90c8e4b9dfa98a41fcd299197a59261be7.tar.gz |
MDEV-22043 Special character leads to assertion in my_wc_to_printable_generic on 10.5.2 (debug)
The code did not take into account that:
- U+005C (backslash) can occupy more than mbminlen characters (e.g. in sjis)
- Some character sets do not have a code for U+005C (e.g. swe7)
Adding a new function my_wc_to_printable into MY_CHARSET_HANDLER to
cover all special cases easier.
Diffstat (limited to 'sql/sql_error.cc')
-rw-r--r-- | sql/sql_error.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sql/sql_error.cc b/sql/sql_error.cc index e5dda9904d2..b3ef0d89a98 100644 --- a/sql/sql_error.cc +++ b/sql/sql_error.cc @@ -850,7 +850,7 @@ extern "C" int my_wc_mb_utf8_null_terminated(CHARSET_INFO *cs, my_wc_t wc, uchar *r, uchar *e) { return wc == '\0' ? - my_wc_to_printable_generic(cs, wc, r, e) : + cs->wc_to_printable(wc, r, e) : my_charset_utf8mb3_handler.wc_mb(cs, wc, r, e); } @@ -951,7 +951,7 @@ size_t convert_error_message(char *to, size_t to_length, CHARSET_INFO *to_cs, to_cs= system_charset_info; uint32 cnv_length= my_convert_using_func(to, to_length, to_cs, - my_wc_to_printable_generic, + to_cs->cset->wc_to_printable, from, from_length, from_cs, from_cs->cset->mb_wc, errors); |