git am with MIME

There's still a problem with git am on the current next branch (which includes the recent "mailinfo: re-fix MIME multipart boundary parsing" patch):

$ wcat -q
'http://article.gmane.org/gmane.comp.version-control.git/91305/raw' | git am
fatal: `pos + len' is too far after the end of the buffer
$

It seems to be because of the (unusual?) way the patch uses MIME. Just wanted to post this so it doesn't get lost.

-- Lea

It's the From header actually. The patch below should fix it (though it sure makes that line of code ugly -- improvements are welcome).

-- >8 --
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 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".

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).

---
 builtin-mailinfo.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

index 26d3e5d..e890f7a 100644
@@ -107,7 +107,7 @@ static void handle_from(const struct strbuf *from)
 	el = strcspn(at, " \n\t\r\v\f>");
 	strbuf_reset(&email);
 	strbuf_add(&email, at, el);
-	strbuf_remove(&f, at - f.buf, el + 1);
+	strbuf_remove(&f, at - f.buf, el + (at[el] ? 1 : 0));
 
 	/* The remainder is name.  It could be "John Doe <john.doe@xz>"
 	 * or "john.doe@xz (John Doe)", but we have removed the

1.6.0.96.g2fad1.dirty

Jeff King, Tue, Aug 19, 2008 19:07:31 +0200:

Ewwww. ;)

-Peff

And why shouldn't they when:

	strbuf_remove(&f, at - f.buf, el + (at[el] != 0));

is infinitely better in every possible way? ;)

2008/8/21 Eric Raible <[email protected]>:

Because that's just as ugly as "?:", brackets and all