mirror of
https://github.com/php/php-src.git
synced 2026-03-27 01:32:22 +01:00
MFB: Fixed bug #44836 (putenv() crashes, avoid direct reference of environ
in POSIX systems).
This commit is contained in:
@@ -3822,9 +3822,7 @@ static void php_putenv_destructor(putenv_entry *pe) /* {{{ */
|
||||
SetEnvironmentVariable(pe->key, "bugbug");
|
||||
#endif
|
||||
putenv(pe->previous_value);
|
||||
# if defined(PHP_WIN32)
|
||||
efree(pe->previous_value);
|
||||
# endif
|
||||
} else {
|
||||
# if HAVE_UNSETENV
|
||||
unsetenv(pe->key);
|
||||
@@ -4388,12 +4386,8 @@ PHP_FUNCTION(putenv)
|
||||
pe.previous_value = NULL;
|
||||
for (env = environ; env != NULL && *env != NULL; env++) {
|
||||
if (!strncmp(*env, pe.key, pe.key_len) && (*env)[pe.key_len] == '=') { /* found it */
|
||||
#if defined(PHP_WIN32)
|
||||
/* must copy previous value because MSVCRT's putenv can free the string without notice */
|
||||
/* must copy previous value because putenv can free the string without notice */
|
||||
pe.previous_value = estrdup(*env);
|
||||
#else
|
||||
pe.previous_value = *env;
|
||||
#endif
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user