> I noticed this the other day, just never got a chance to send the fix out.
> This might be the same problem I ran into.
>
> Cheers,
> Don
>
> builtin-mailinfo.c | 5 ++++-
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/builtin-mailinfo.c b/builtin-mailinfo.c
> index 2894e34..cedda18 100644
> --- a/builtin-mailinfo.c
> +++ b/builtin-mailinfo.c
> @@ -795,7 +795,7 @@ static void handle_body(void)
> int rc = 0;
> static char newline[2000];
> static char *np = newline;
> - int len = strlen(line);
> + int len;
>
> /* Skip up to the first boundary */
> if (content_top->boundary) {
> @@ -814,6 +814,9 @@ static void handle_body(void)
> return;
> }
>
> + /* line may have changed after handling boundary, check len */
> + len = strlen(line);
> +
> /* Unwrap transfer encoding */
> len = decode_transfer_encoding(line, sizeof(line), len);
> if (len < 0) {
This does fix the "F\n" issue, but seems to break t5100 test ("respect
NULs"). I haven't looked into the details yet...
--
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
