It is not that uncommon to have mails with DOS line-ending, notably Thunderbird and Gmail (when saving what they call "original" message). Noticed by H. Peter Anvin. Signed-off-by: Alex Riesen <raa.lkml@xxxxxxxxx> --- Alex Riesen <raa.lkml@xxxxxxxxx> writes: > Maybe it is as simple as that (not tested yet, > and sent through gmail, so please be careful): > --- a/builtin-mailsplit.c > +++ b/builtin-mailsplit.c > @@ -58,6 +58,8 @@ int read_line_with_nul(char *buf, int size, FILE *in) > + if (len && buf[len - 1] == '\r') > + --len; That's wrong, of course. I missed the fact that \n stays in the buffer. Corrected. builtin-mailsplit.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/builtin-mailsplit.c b/builtin-mailsplit.c index ad5f6b5..188ffec 100644 --- a/builtin-mailsplit.c +++ b/builtin-mailsplit.c @@ -58,6 +58,8 @@ int read_line_with_nul(char *buf, int size, FILE *in) if (c == '\n' || len + 1 >= size) break; } + if (len > 1 && buf[len - 2] == '\r') + buf[--len - 1] = '\n'; buf[len] = '\0'; return len; -- 1.6.4.30.gd4c0 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html