path: root/t/t5100
diff options
authorJeff King <>2008-08-19 17:28:24 (GMT)
committerJunio C Hamano <>2008-08-20 02:36:56 (GMT)
commite9d7d10a7f17fb9cc6a4d37b6fdf27170f4deede (patch)
treefd6d405f55a76858d0437a3fba201c17b8a0d975 /t/t5100
parentc8c4450e1949055cb57e32425b125f45f3481742 (diff)
mailinfo: avoid violating strbuf assertion
In handle_from, we calculate the end boundary of a section to remove from a strbuf using strcspn like this: el = strcspn(buf, set_of_end_boundaries); strbuf_remove(&sb, start, el + 1); This works fine if "el" is the offset of the boundary character, meaning we remove up to and including that character. But if the end boundary didn't match (that is, we hit the end of the string as the boundary instead) then we want just "el". Asking for "el+1" caught an out-of-bounds assertion in the strbuf library. This manifested itself when we got a 'From' header that had just an email address with nothing else in it (the end of the string was the end of the address, rather than, e.g., a trailing '>' character), causing git-mailinfo to barf. Signed-off-by: Jeff King <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 't/t5100')
2 files changed, 13 insertions, 0 deletions
diff --git a/t/t5100/info-from.expect b/t/t5100/info-from.expect
new file mode 100644
index 0000000..c31d2eb
--- /dev/null
+++ b/t/t5100/info-from.expect
@@ -0,0 +1,5 @@
+Subject: testing bare address in from header
+Date: Sun, 25 May 2008 00:38:18 -0700
diff --git a/t/t5100/ b/t/t5100/
new file mode 100644
index 0000000..4f08209
--- /dev/null
+++ b/t/t5100/
@@ -0,0 +1,8 @@
+From 667d8940e719cddee1cfe237cbbe215e20270b09 Mon Sep 17 00:00:00 2001
+Date: Sun, 25 May 2008 00:38:18 -0700
+Subject: [PATCH] testing bare address in from header
+commit message