wildcard_cmp.c: Fix a couple of bugs
wildcard_cmp would otherwise fail with the following use cases: s = "mymi", p = "*mi*" s = "mymi", p = "*mi"
This commit is contained in:
parent
0d7ead4638
commit
6adf1c44ad
|
@ -6,6 +6,7 @@
|
||||||
|
|
||||||
int wildcard_cmp(const char *s, const char *p, const bool casecmp)
|
int wildcard_cmp(const char *s, const char *p, const bool casecmp)
|
||||||
{
|
{
|
||||||
|
const char *last = NULL;
|
||||||
int (*const cmp)(const char *, const char *) =
|
int (*const cmp)(const char *, const char *) =
|
||||||
casecmp ? strcmp : strcasecmp;
|
casecmp ? strcmp : strcasecmp;
|
||||||
int (*const ncmp)(const char *, const char *, size_t) =
|
int (*const ncmp)(const char *, const char *, size_t) =
|
||||||
|
@ -16,7 +17,18 @@ int wildcard_cmp(const char *s, const char *p, const bool casecmp)
|
||||||
const char *const wc = strchr(p, '*');
|
const char *const wc = strchr(p, '*');
|
||||||
|
|
||||||
if (!wc)
|
if (!wc)
|
||||||
return cmp(s, p);
|
{
|
||||||
|
const int r = cmp(s, p);
|
||||||
|
|
||||||
|
if (r && last)
|
||||||
|
{
|
||||||
|
p = last;
|
||||||
|
s += strlen(p);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
else
|
||||||
|
return r;
|
||||||
|
}
|
||||||
|
|
||||||
const size_t n = wc - p;
|
const size_t n = wc - p;
|
||||||
|
|
||||||
|
@ -25,9 +37,15 @@ int wildcard_cmp(const char *s, const char *p, const bool casecmp)
|
||||||
const int r = ncmp(s, p, n);
|
const int r = ncmp(s, p, n);
|
||||||
|
|
||||||
if (r)
|
if (r)
|
||||||
return r;
|
{
|
||||||
|
if (last)
|
||||||
|
p = last;
|
||||||
|
else
|
||||||
|
return r;
|
||||||
|
}
|
||||||
|
else
|
||||||
|
p += n;
|
||||||
|
|
||||||
p += n;
|
|
||||||
s += n;
|
s += n;
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
|
@ -36,6 +54,7 @@ int wildcard_cmp(const char *s, const char *p, const bool casecmp)
|
||||||
|
|
||||||
if (!cmp(wca, sa))
|
if (!cmp(wca, sa))
|
||||||
{
|
{
|
||||||
|
last = p;
|
||||||
p = wc + 1;
|
p = wc + 1;
|
||||||
}
|
}
|
||||||
else if (next == '*')
|
else if (next == '*')
|
||||||
|
|
Loading…
Reference in New Issue