[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH] sendmail: Fix parsing of addresses in angle brackets
From:       Denys Vlasenko <vda.linux () googlemail ! com>
Date:       2018-07-04 14:18:58
Message-ID: CAK1hOcMQMmSf=K29wfoOjyDR5EQWPx6HW=D2FKgWu8pTGkxFEQ () mail ! gmail ! com
[Download RAW message or body]

applied, thanks!

On Wed, Jun 27, 2018 at 1:18 AM, Raffaello D. Di Napoli
<rafdev@dinapo.li> wrote:
> Pointer e needs to be wound back in order to overwrite '>' with '\0'.
> Regression introduced in 20077c1429 .
>
> Signed-off-by: Raffaello D. Di Napoli <rafdev@dinapo.li>
> ---
>  mailutils/sendmail.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/mailutils/sendmail.c b/mailutils/sendmail.c
> index 804e53a33..0441528ab 100644
> --- a/mailutils/sendmail.c
> +++ b/mailutils/sendmail.c
> @@ -173,7 +173,7 @@ static char *angle_address(char *str)
>         char *s, *e;
>
>         e = trim(str);
> -       if (e != str && e[-1] == '>') {
> +       if (e != str && *--e == '>') {
>                 s = strrchr(str, '<');
>                 if (s) {
>                         *e = '\0';
> --
> 2.16.4
>
> _______________________________________________
> busybox mailing list
> busybox@busybox.net
> http://lists.busybox.net/mailman/listinfo/busybox
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]