[git commit] sendmail: fix segfault if "To: email1,email2" is used

Denys Vlasenko vda.linux at googlemail.com
Tue Aug 22 13:53:16 UTC 2017


commit: https://git.busybox.net/busybox/commit/?id=3505e38bd1e5714fa1203e6752c573861aab8521
branch: https://git.busybox.net/busybox/commit/?id=refs/heads/master

Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
 mailutils/sendmail.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/mailutils/sendmail.c b/mailutils/sendmail.c
index f440e63..defbd95 100644
--- a/mailutils/sendmail.c
+++ b/mailutils/sendmail.c
@@ -189,8 +189,9 @@ static void rcptto(const char *s)
 // send to a list of comma separated addresses
 static void rcptto_list(const char *list)
 {
-	char *str = xstrdup(list);
-	char *s = str;
+	char *free_me = xstrdup(list);
+	char *str = free_me;
+	char *s = free_me;
 	char prev = 0;
 	int in_quote = 0;
 
@@ -208,7 +209,7 @@ static void rcptto_list(const char *list)
 	}
 	if (prev != ',')
 		rcptto(angle_address(str));
-	free(str);
+	free(free_me);
 }
 
 int sendmail_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;


More information about the busybox-cvs mailing list