diff options
author | ghelmer <ghelmer@FreeBSD.org> | 2000-04-28 15:31:28 +0000 |
---|---|---|
committer | ghelmer <ghelmer@FreeBSD.org> | 2000-04-28 15:31:28 +0000 |
commit | 5869eff8901701f3215b4a78477cc57e21f16416 (patch) | |
tree | 456b0ef2c04594f53600bf8d6955aefa0bd3cf53 /usr.sbin/cron | |
parent | 68665ff50512fd9baa5bc783a90faac11670b0dd (diff) | |
download | FreeBSD-src-5869eff8901701f3215b4a78477cc57e21f16416.zip FreeBSD-src-5869eff8901701f3215b4a78477cc57e21f16416.tar.gz |
Fix a situation where a pointer which should point to dynamically
allocated memory was instead pointed to a static string. A later
free() on the value of the pointer was a possible source of reported
"warning: pointer to wrong page" messages from cron.
Use consistent types in sizeof when malloc'ing memory for the
environment.
PR: kern/12248, bin/11169, bin/9722
Diffstat (limited to 'usr.sbin/cron')
-rw-r--r-- | usr.sbin/cron/lib/env.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/usr.sbin/cron/lib/env.c b/usr.sbin/cron/lib/env.c index c5f2200..eb956a0 100644 --- a/usr.sbin/cron/lib/env.c +++ b/usr.sbin/cron/lib/env.c @@ -56,7 +56,7 @@ env_copy(envp) for (count = 0; envp[count] != NULL; count++) ; - p = (char **) malloc((count+1) * sizeof(char *)); /* 1 for the NULL */ + p = (char **) malloc((count+1) * sizeof(char **)); /* 1 for the NULL */ if (p == NULL) { errno = ENOMEM; return NULL; @@ -81,6 +81,7 @@ env_set(envp, envstr) { register int count, found; register char **p; + char *q; /* * count the number of elements, including the null pointer; @@ -98,12 +99,14 @@ env_set(envp, envstr) * it exists already, so just free the existing setting, * save our new one there, and return the existing array. */ - free(envp[found]); + q = envp[found]; if ((envp[found] = strdup(envstr)) == NULL) { - envp[found] = ""; + envp[found] = q; + /* XXX env_free(envp); */ errno = ENOMEM; return NULL; } + free(q); return (envp); } @@ -115,11 +118,13 @@ env_set(envp, envstr) p = (char **) realloc((void *) envp, (unsigned) ((count+1) * sizeof(char **))); if (p == NULL) { + /* XXX env_free(envp); */ errno = ENOMEM; return NULL; } p[count] = p[count-1]; if ((p[count-1] = strdup(envstr)) == NULL) { + env_free(p); errno = ENOMEM; return NULL; } |