diff options
| author |  Don Zickus <dzickus@redhat.com> | 2008-08-14 11:35:42 -0400 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2008-08-18 22:05:02 -0700 |
| commit | 289796dd29dd656734cfd59b657deb943a71cf6a (patch) | |
| tree | 7ff6e57895095b4ef499bd18c8d665e43a457103 /t/t5100 | |
| parent | Start 1.6.0.X maintenance series (diff) | |
| download | tgif-289796dd29dd656734cfd59b657deb943a71cf6a.tar.xz | |
mailinfo: re-fix MIME multipart boundary parsing
Recent changes to is_multipart_boundary() caused git-mailinfo to segfault.
The reason was after handling the end of the boundary the code tried to look
for another boundary. Because the boundary list was empty, dereferencing
the pointer to the top of the boundary caused the program to go boom.
The fix is to check to see if the list is empty and if so go on its merry
way instead of looking for another boundary.
I also fixed a couple of increments and decrements that didn't look correct
relating to content_top.
The boundary test case was updated to catch future problems like this again.
Signed-off-by: Don Zickus <dzickus@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t5100')
| -rw-r--r-- | t/t5100/sample.mbox | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/t/t5100/sample.mbox b/t/t5100/sample.mbox index d7ca79b1fc..4bf7947b41 100644 --- a/t/t5100/sample.mbox +++ b/t/t5100/sample.mbox @@ -500,3 +500,4 @@ index 3e5fe51..aabfe5c 100644 1.6.0.rc2 --=-=-=-- + |