93 lines
3.0 KiB
Diff
93 lines
3.0 KiB
Diff
|
When fnmatch detects an invalid multibyte character it should fall back to
|
||
|
single byte matching, so that "*" has a chance to match such a string.
|
||
|
|
||
|
Andreas.
|
||
|
|
||
|
Ported to glibc-2.11.3 by Petr Baudis.
|
||
|
|
||
|
2005-04-12 Andreas Schwab <schwab@suse.de>
|
||
|
|
||
|
* posix/fnmatch.c (fnmatch): If conversion to wide character
|
||
|
fails fall back to single byte matching.
|
||
|
|
||
|
Index: glibc-2.11.2/posix/fnmatch.c
|
||
|
===================================================================
|
||
|
--- glibc-2.11.2.orig/posix/fnmatch.c
|
||
|
+++ glibc-2.11.2/posix/fnmatch.c
|
||
|
@@ -333,6 +333,7 @@ fnmatch (pattern, string, flags)
|
||
|
# if HANDLE_MULTIBYTE
|
||
|
if (__builtin_expect (MB_CUR_MAX, 1) != 1)
|
||
|
{
|
||
|
+ const char *orig_pattern = pattern;
|
||
|
mbstate_t ps;
|
||
|
size_t n;
|
||
|
const char *p;
|
||
|
@@ -356,10 +357,8 @@ fnmatch (pattern, string, flags)
|
||
|
alloca_used);
|
||
|
n = mbsrtowcs (wpattern, &p, n + 1, &ps);
|
||
|
if (__builtin_expect (n == (size_t) -1, 0))
|
||
|
- /* Something wrong.
|
||
|
- XXX Do we have to set `errno' to something which mbsrtows hasn't
|
||
|
- already done? */
|
||
|
- return -1;
|
||
|
+ /* Something wrong. Fall back to single byte matching. */
|
||
|
+ goto try_singlebyte;
|
||
|
if (p)
|
||
|
{
|
||
|
memset (&ps, '\0', sizeof (ps));
|
||
|
@@ -371,10 +370,8 @@ fnmatch (pattern, string, flags)
|
||
|
prepare_wpattern:
|
||
|
n = mbsrtowcs (NULL, &pattern, 0, &ps);
|
||
|
if (__builtin_expect (n == (size_t) -1, 0))
|
||
|
- /* Something wrong.
|
||
|
- XXX Do we have to set `errno' to something which mbsrtows hasn't
|
||
|
- already done? */
|
||
|
- return -1;
|
||
|
+ /* Something wrong. Fall back to single byte matching. */
|
||
|
+ goto try_singlebyte;
|
||
|
wpattern_malloc = wpattern
|
||
|
= (wchar_t *) malloc ((n + 1) * sizeof (wchar_t));
|
||
|
assert (mbsinit (&ps));
|
||
|
@@ -396,14 +393,8 @@ fnmatch (pattern, string, flags)
|
||
|
alloca_used);
|
||
|
n = mbsrtowcs (wstring, &p, n + 1, &ps);
|
||
|
if (__builtin_expect (n == (size_t) -1, 0))
|
||
|
- {
|
||
|
- /* Something wrong.
|
||
|
- XXX Do we have to set `errno' to something which
|
||
|
- mbsrtows hasn't already done? */
|
||
|
- free_return:
|
||
|
- free (wpattern_malloc);
|
||
|
- return -1;
|
||
|
- }
|
||
|
+ /* Something wrong. Fall back to single byte matching. */
|
||
|
+ goto free_and_try_singlebyte;
|
||
|
if (p)
|
||
|
{
|
||
|
memset (&ps, '\0', sizeof (ps));
|
||
|
@@ -415,10 +406,8 @@ fnmatch (pattern, string, flags)
|
||
|
prepare_wstring:
|
||
|
n = mbsrtowcs (NULL, &string, 0, &ps);
|
||
|
if (__builtin_expect (n == (size_t) -1, 0))
|
||
|
- /* Something wrong.
|
||
|
- XXX Do we have to set `errno' to something which mbsrtows hasn't
|
||
|
- already done? */
|
||
|
- goto free_return;
|
||
|
+ /* Something wrong. Fall back to single byte matching. */
|
||
|
+ goto free_and_try_singlebyte;
|
||
|
|
||
|
wstring_malloc = wstring
|
||
|
= (wchar_t *) malloc ((n + 1) * sizeof (wchar_t));
|
||
|
@@ -439,6 +428,11 @@ fnmatch (pattern, string, flags)
|
||
|
free (wpattern_malloc);
|
||
|
|
||
|
return res;
|
||
|
+
|
||
|
+ free_and_try_singlebyte:
|
||
|
+ free(wpattern_malloc);
|
||
|
+ try_singlebyte:
|
||
|
+ pattern = orig_pattern;
|
||
|
}
|
||
|
# endif /* mbstate_t and mbsrtowcs or _LIBC. */
|
||
|
|