diff options
| author | Junio C Hamano <gitster@pobox.com> | 2023-12-27 22:52:24 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2023-12-27 22:52:24 (GMT) |
| commit | db2cf6f3bbab88631d98ec737591b52ae4d87251 (patch) | |
| tree | 061516d04736d609fbd6857b0a9f1ff1e1e401f4 /mailinfo.c | |
| parent | f09e74175dfa5b46587dab3c5dcbad16e9586de3 (diff) | |
| parent | d1bd3a8c3424e818f4117a39fe418909e24cea5f (diff) | |
| download | git-db2cf6f3bbab88631d98ec737591b52ae4d87251.zip git-db2cf6f3bbab88631d98ec737591b52ae4d87251.tar.gz git-db2cf6f3bbab88631d98ec737591b52ae4d87251.tar.bz2 | |
Merge branch 'jk/mailinfo-oob-read-fix'
OOB read fix.
* jk/mailinfo-oob-read-fix:
mailinfo: fix out-of-bounds memory reads in unquote_quoted_pair()
Diffstat (limited to 'mailinfo.c')
| -rw-r--r-- | mailinfo.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -58,12 +58,12 @@ static void parse_bogus_from(struct mailinfo *mi, const struct strbuf *line) static const char *unquote_comment(struct strbuf *outbuf, const char *in) { - int c; int take_next_literally = 0; strbuf_addch(outbuf, '('); - while ((c = *in++) != 0) { + while (*in) { + int c = *in++; if (take_next_literally == 1) { take_next_literally = 0; } else { @@ -88,10 +88,10 @@ static const char *unquote_comment(struct strbuf *outbuf, const char *in) static const char *unquote_quoted_string(struct strbuf *outbuf, const char *in) { - int c; int take_next_literally = 0; - while ((c = *in++) != 0) { + while (*in) { + int c = *in++; if (take_next_literally == 1) { take_next_literally = 0; } else { |