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

List:       busybox
Subject:    [PATCH] sendmail: Fix parsing of addresses in angle brackets
From:       "Raffaello D. Di Napoli" <rafdev () dinapo ! li>
Date:       2018-06-26 23:18:02
Message-ID: 20180626231802.7392020432 () mail ! rd2n ! net
[Download RAW message or body]

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
[prev in list] [next in list] [prev in thread] [next in thread]