diff options
author | Peter Stephenson <pws@zsh.org> | 2015-10-01 16:21:18 +0100 |
---|---|---|
committer | Peter Stephenson <pws@zsh.org> | 2015-10-01 16:21:18 +0100 |
commit | 807a8338a3be8127dd23c69971668b7b0c6b79a2 (patch) | |
tree | 8c1c3cc0f3b9005f950138153ba549c59fd90c48 /Src | |
parent | 196e6a0ec022fb2fc22820a346b5f206a3cf40ac (diff) | |
download | zsh-807a8338a3be8127dd23c69971668b7b0c6b79a2.tar.gz zsh-807a8338a3be8127dd23c69971668b7b0c6b79a2.tar.xz zsh-807a8338a3be8127dd23c69971668b7b0c6b79a2.zip |
36737: Ensure we don't dreference unterminated zero-length string
Diffstat (limited to 'Src')
-rw-r--r-- | Src/pattern.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/Src/pattern.c b/Src/pattern.c index 68a340919..04d3e3dfb 100644 --- a/Src/pattern.c +++ b/Src/pattern.c @@ -2224,8 +2224,10 @@ pattryrefs(Patprog prog, char *string, int stringlen, int unmetalenin, maxnpos = *nump; *nump = 0; } - /* inherited from domatch, but why, exactly? */ - if (*string == Nularg) { + /* + * Special signalling of empty tokenised string. + */ + if ((!patstralloc || stringlen > 0) && *string == Nularg) { string++; if (unmetalenin > 0) unmetalenin--; @@ -2233,8 +2235,10 @@ pattryrefs(Patprog prog, char *string, int stringlen, int unmetalenin, stringlen--; } - if (stringlen < 0) + if (stringlen < 0) { + DPUTS(patstralloc != NULL, "length needed with patstralloc"); stringlen = strlen(string); + } origlen = stringlen; if (patstralloc) { |