From: Edin Kadribasic Date: Mon, 15 Oct 2001 23:30:40 +0000 (+0000) Subject: Fixed bug #13083 in http_fopen_wrapper.c which is now able to handle X-Git-Tag: POST_PARAMETER_PARSING_API~104 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=96a04d4ab7b56094c658a88c270a3fe6c861dd7e;p=php Fixed bug #13083 in http_fopen_wrapper.c which is now able to handle non absolute redirects. --- diff --git a/ext/standard/http_fopen_wrapper.c b/ext/standard/http_fopen_wrapper.c index 83c6f5dc1d..9a4132d712 100644 --- a/ext/standard/http_fopen_wrapper.c +++ b/ext/standard/http_fopen_wrapper.c @@ -271,11 +271,28 @@ FILE *php_fopen_url_wrap_http(const char *path, char *mode, int options, int *is if (!reqok) { SOCK_FCLOSE(*socketd); *socketd = 0; - php_url_free(resource); if (location[0] != '\0') { zval **response_header_new, *entry, **entryp; + char new_path[512]; - fp = php_fopen_url_wrap_http(location, mode, options, issock, socketd, opened_path TSRMLS_CC); + *new_path='\0'; + if (strlen(location)<8 || strncasecmp(location, "http://", 7)) { + strcpy(new_path, "http://"); + strncat(new_path, resource->host, sizeof(new_path)-strlen(new_path)-1); + if (resource->port != 80) { + snprintf(new_path+strlen(new_path), sizeof(new_path)-strlen(new_path)-1, ":%d", resource->port); + } + if (*location != '/') { + php_dirname(resource->path, strlen(resource->path)); + snprintf (new_path+strlen(new_path), sizeof(new_path)-strlen(new_path)-1, "%s/", resource->path); + } + strncat(new_path, location, sizeof(new_path)-strlen(new_path)-1); + } + else { + strncpy(new_path, location, sizeof(new_path)); + } + php_url_free(resource); + fp = php_fopen_url_wrap_http(new_path, mode, options, issock, socketd, opened_path TSRMLS_CC); if (zend_hash_find(EG(active_symbol_table), "http_response_header", sizeof("http_response_header"), (void **) &response_header_new) == SUCCESS) { entryp = &entry; MAKE_STD_ZVAL(entry); @@ -290,6 +307,7 @@ FILE *php_fopen_url_wrap_http(const char *path, char *mode, int options, int *is } goto out; } else { + php_url_free(resource); fp = NULL; goto out; }