From af957f2ed6287f66953742fbca69188cecb98fbf Mon Sep 17 00:00:00 2001 From: "Barton E. Schaefer" Date: Wed, 20 May 2015 10:14:04 -0700 Subject: 35231: make mkevnstr() safe for NULL value --- ChangeLog | 4 ++++ Src/params.c | 14 +++++++++----- 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/ChangeLog b/ChangeLog index 0bdeeaeb3..b1b96379c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -3,6 +3,10 @@ * Ismail: 35232: Completion/Unix/Type/_urls: matching parentheses. +2015-05-20 Barton E. Schaefer + + * 35231: Src/params.c: make mkevnstr() safe for NULL value + 2015-05-19 Daniel Shahaf * 35224: Completion/Unix/Command/_git: completion: git: Add diff --git a/Src/params.c b/Src/params.c index 045ac1e34..98541a6da 100644 --- a/Src/params.c +++ b/Src/params.c @@ -4580,17 +4580,21 @@ addenv(Param pm, char *value) static char * mkenvstr(char *name, char *value, int flags) { - char *str, *s; - int len_name, len_value; + char *str, *s = value; + int len_name, len_value = 0; len_name = strlen(name); - for (len_value = 0, s = value; - *s && (*s++ != Meta || *s++ != 32); len_value++); + if (s) + while (*s && (*s++ != Meta || *s++ != 32)) + len_value++; s = str = (char *) zalloc(len_name + len_value + 2); strcpy(s, name); s += len_name; *s = '='; - copyenvstr(s, value, flags); + if (value) + copyenvstr(s, value, flags); + else + *++s = '\0'; return str; } -- cgit 1.4.1