diff options
| -rw-r--r-- | ChangeLog | 7 | ||||
| -rw-r--r-- | posix/fnmatch.c | 42 | ||||
| -rw-r--r-- | posix/testfnm.c | 3 |
3 files changed, 39 insertions, 13 deletions
diff --git a/ChangeLog b/ChangeLog index 38a0b6dfc5..200c811dda 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +1999-04-28 Ulrich Drepper <drepper@cygnus.com> + + * posix/fnmatch.c (internal_fnmatch): Finish incomplete + no_leading_period change. + + * posix/testfnm.c: More test cases. + 1999-04-27 Ulrich Drepper <drepper@cygnus.com> * include/dirent.h: Remove __getdirentries declaration, add __getdents diff --git a/posix/fnmatch.c b/posix/fnmatch.c index b147f47d36..eb3f1cc2d4 100644 --- a/posix/fnmatch.c +++ b/posix/fnmatch.c @@ -157,7 +157,9 @@ internal_fnmatch (const char *pattern, const char *string, return FNM_NOMATCH; else if (*n == '/' && (flags & FNM_FILE_NAME)) return FNM_NOMATCH; - else if (*n == '.' && no_leading_period) + else if (*n == '.' && no_leading_period + && (n == string + || (n[-1] == '/' && (flags & FNM_FILE_NAME)))) return FNM_NOMATCH; break; @@ -175,7 +177,9 @@ internal_fnmatch (const char *pattern, const char *string, break; case '*': - if (*n == '.' && no_leading_period) + if (*n == '.' && no_leading_period + && (n == string + || (n[-1] == '/' && (flags & FNM_FILE_NAME)))) return FNM_NOMATCH; for (c = *p++; c == '?' || c == '*'; c = *p++) @@ -212,11 +216,17 @@ internal_fnmatch (const char *pattern, const char *string, if (c == '[') { + int flags2 = ((flags & FNM_FILE_NAME) + ? flags : (flags & ~FNM_PERIOD)); + for (--p; n < endp; ++n) if (internal_fnmatch (p, n, - (n == string) && no_leading_period, - ((flags & FNM_FILE_NAME) - ? flags : (flags & ~FNM_PERIOD))) + (no_leading_period + && (n == string + || (n[-1] == '/' + && (flags + & FNM_FILE_NAME)))), + flags2) == 0) return 0; } @@ -231,18 +241,21 @@ internal_fnmatch (const char *pattern, const char *string, } else { + int flags2 = ((flags & FNM_FILE_NAME) + ? flags : (flags & ~FNM_PERIOD)); + if (c == '\\' && !(flags & FNM_NOESCAPE)) c = *p; c = FOLD (c); for (--p; n < endp; ++n) if (FOLD ((unsigned char) *n) == c && (internal_fnmatch (p, n, - ((n == string) - && no_leading_period), - ((flags & FNM_FILE_NAME) - ? flags - : (flags & ~FNM_PERIOD))) - == 0)) + (no_leading_period + && (n == string + || (n[-1] == '/' + && (flags + & FNM_FILE_NAME)))), + flags2) == 0)) return 0; } } @@ -263,7 +276,10 @@ internal_fnmatch (const char *pattern, const char *string, if (*n == '\0') return FNM_NOMATCH; - if (*n == '.' && no_leading_period) + if (*n == '.' && no_leading_period && (n == string + || (n[-1] == '/' + && (flags + & FNM_FILE_NAME)))) return FNM_NOMATCH; if (*n == '/' && (flags & FNM_FILE_NAME)) @@ -297,7 +313,7 @@ internal_fnmatch (const char *pattern, const char *string, # if defined _LIBC || (defined HAVE_WCTYPE_H && defined HAVE_WCHAR_H) wctype_t wt; # endif - char *startp = p; + const char *startp = p; for (;;) { diff --git a/posix/testfnm.c b/posix/testfnm.c index e300565b1b..ebdfab41de 100644 --- a/posix/testfnm.c +++ b/posix/testfnm.c @@ -48,6 +48,9 @@ struct { { "a/b", "*[![:digit:]]*/[![:d-d]", FNM_PATHNAME, 0 }, { "a/[", "*[![:digit:]]*/[[:d-d]", FNM_PATHNAME, 0 }, { "a/[", "*[![:digit:]]*/[![:d-d]", FNM_PATHNAME, FNM_NOMATCH }, + { "a.b", "a?b", FNM_PATHNAME|FNM_PERIOD, 0 }, + { "a.b", "a*b", FNM_PATHNAME|FNM_PERIOD, 0 }, + { "a.b", "a[.]b", FNM_PATHNAME|FNM_PERIOD, 0 }, }; int |