path: root/mailmap.c
diff options
authorRomain Francoise <>2012-10-27 22:49:55 (GMT)
committerJeff King <>2012-10-28 11:50:18 (GMT)
commit3174bc5ccfea34717a8dac2175c1951239a4985d (patch)
tree401f10be8b7f4bf84dc9ed7c46774fb4f0fbd650 /mailmap.c
parent7e2010537e96d0a1144520222f20ba1dc3d61441 (diff)
mailmap: avoid out-of-bounds memory access
AddressSanitizer ( complains of a one-byte buffer underflow in parse_name_and_email() while running the test suite. And indeed, if one of the lines in the mailmap begins with '<', we dereference the address just before the beginning of the buffer when looking for whitespace to remove, before checking that we aren't going too far. So reverse the order of the tests to make sure that we don't read outside the buffer. Signed-off-by: Romain Francoise <> Signed-off-by: Jeff King <>
Diffstat (limited to 'mailmap.c')
1 files changed, 1 insertions, 1 deletions
diff --git a/mailmap.c b/mailmap.c
index 47aa419..ea4b471 100644
--- a/mailmap.c
+++ b/mailmap.c
@@ -118,7 +118,7 @@ static char *parse_name_and_email(char *buffer, char **name,
while (isspace(*nstart) && nstart < left)
nend = left-1;
- while (isspace(*nend) && nend > nstart)
+ while (nend > nstart && isspace(*nend))
*name = (nstart < nend ? nstart : NULL);