mirror of
https://github.com/php/php-src.git
synced 2024-11-24 18:34:21 +08:00
MFH: fix segfault - error_string may be NULL, in this case we have to follow the old behavior
This commit is contained in:
parent
e897db28bf
commit
f0b9ac513c
@ -205,12 +205,20 @@ static int php_network_getaddresses(const char *host, int socktype, struct socka
|
||||
# endif
|
||||
|
||||
if ((n = getaddrinfo(host, NULL, &hints, &res))) {
|
||||
spprintf(error_string, 0, "php_network_getaddresses: getaddrinfo failed: %s", PHP_GAI_STRERROR(n));
|
||||
php_error_docref(NULL TSRMLS_CC, E_WARNING, "%s", *error_string);
|
||||
if (error_string) {
|
||||
spprintf(error_string, 0, "php_network_getaddresses: getaddrinfo failed: %s", PHP_GAI_STRERROR(n));
|
||||
php_error_docref(NULL TSRMLS_CC, E_WARNING, "%s", *error_string);
|
||||
} else {
|
||||
php_error_docref(NULL TSRMLS_CC, E_WARNING, "php_network_getaddresses: getaddrinfo failed: %s", PHP_GAI_STRERROR(n));
|
||||
}
|
||||
return 0;
|
||||
} else if (res == NULL) {
|
||||
spprintf(error_string, 0, "php_network_getaddresses: getaddrinfo failed (null result pointer) errno=%d", errno);
|
||||
php_error_docref(NULL TSRMLS_CC, E_WARNING, "%s", *error_string);
|
||||
if (error_string) {
|
||||
spprintf(error_string, 0, "php_network_getaddresses: getaddrinfo failed (null result pointer) errno=%d", errno);
|
||||
php_error_docref(NULL TSRMLS_CC, E_WARNING, "%s", *error_string);
|
||||
} else {
|
||||
php_error_docref(NULL TSRMLS_CC, E_WARNING, "php_network_getaddresses: getaddrinfo failed (null result pointer)");
|
||||
}
|
||||
return 0;
|
||||
}
|
||||
|
||||
@ -234,8 +242,12 @@ static int php_network_getaddresses(const char *host, int socktype, struct socka
|
||||
/* XXX NOT THREAD SAFE (is safe under win32) */
|
||||
host_info = gethostbyname(host);
|
||||
if (host_info == NULL) {
|
||||
spprintf(error_string, 0, "php_network_getaddresses: gethostbyname failed. errno=%d", errno);
|
||||
php_error_docref(NULL TSRMLS_CC, E_WARNING, "%s", *error_string);
|
||||
if (error_string) {
|
||||
spprintf(error_string, 0, "php_network_getaddresses: gethostbyname failed. errno=%d", errno);
|
||||
php_error_docref(NULL TSRMLS_CC, E_WARNING, "%s", *error_string);
|
||||
} else {
|
||||
php_error_docref(NULL TSRMLS_CC, E_WARNING, "php_network_getaddresses: gethostbyname failed");
|
||||
}
|
||||
return 0;
|
||||
}
|
||||
in = *((struct in_addr *) host_info->h_addr);
|
||||
|
Loading…
Reference in New Issue
Block a user