X-Git-Url: https://git.m6w6.name/?a=blobdiff_plain;f=http_api.c;h=2bc553e2763c4d1211942e86f470fbfff5d0df34;hb=e0b95d0689396baab08049b7de5063ff86a7b62a;hp=7904c5d9271ac295671787dc1fcced2f1b939a52;hpb=8d7fee64c740137b693630ad82374c11d1aaa76a;p=m6w6%2Fext-http

diff --git a/http_api.c b/http_api.c
index 7904c5d..2bc553e 100644
--- a/http_api.c
+++ b/http_api.c
@@ -593,7 +593,7 @@ static inline void _http_curl_setopts(CURL *ch, const char *url, HashTable *opti
 			if (key_type == HASH_KEY_IS_STRING) {
 				zend_hash_get_current_key(Z_ARRVAL_P(zoption), &header_key, NULL, 0);
 				zend_hash_get_current_data(Z_ARRVAL_P(zoption), (void **) &header_val);
-				snprintf(header, 1024, "%s: %s", header_key, Z_STRVAL_PP(header_val));
+				snprintf(header, 1023, "%s: %s", header_key, Z_STRVAL_PP(header_val));
 				headers = curl_slist_append(headers, header);
 				zend_hash_move_forward(Z_ARRVAL_P(zoption));
 			}
@@ -685,7 +685,7 @@ static inline void _http_curl_getinfo_ex(CURL *ch, CURLINFO i, zval *array TSRML
 			{
 				double d;
 				if (CURLE_OK == curl_easy_getinfo(ch, i, &d)) {
-					add_assoc_double(array, key, (double) d);
+					add_assoc_double(array, key, d);
 				}
 			}
 			break;
@@ -694,7 +694,7 @@ static inline void _http_curl_getinfo_ex(CURL *ch, CURLINFO i, zval *array TSRML
 			{
 				long l;
 				if (CURLE_OK == curl_easy_getinfo(ch, i, &l)) {
-					add_assoc_long(array, key, (long) l);
+					add_assoc_long(array, key, l);
 				}
 			}
 			break;
@@ -706,12 +706,10 @@ static inline void _http_curl_getinfo_ex(CURL *ch, CURLINFO i, zval *array TSRML
 /* {{{ static inline http_curl_getinfo(CURL, HashTable *) */
 static inline void _http_curl_getinfo(CURL *ch, HashTable *info TSRMLS_DC)
 {
-	zval *array;
+	zval array;
+	Z_ARRVAL(array) = info;
 
-	MAKE_STD_ZVAL(array);
-	Z_ARRVAL_P(array) = info;
-
-#define INFO(I) http_curl_getinfo_ex(ch, CURLINFO_ ##I , array)
+#define INFO(I) http_curl_getinfo_ex(ch, CURLINFO_ ##I , &array)
 	/* CURLINFO_EFFECTIVE_URL			=	CURLINFO_STRING	+1, */
 	INFO(EFFECTIVE_URL);
 	/* CURLINFO_RESPONSE_CODE			=	CURLINFO_LONG	+2, */
@@ -761,7 +759,6 @@ static inline void _http_curl_getinfo(CURL *ch, HashTable *info TSRMLS_DC)
 	/* CURLINFO_PROXYAUTH_AVAIL			=	CURLINFO_LONG	+24, */
 	INFO(PROXYAUTH_AVAIL);
 #undef INFO
-	efree(array);
 }
 /* }}} */
 
@@ -848,7 +845,7 @@ PHP_HTTP_API char *_http_date(time_t t TSRMLS_DC)
 	char *date = ecalloc(1, 30);
 
 	gmtime = php_gmtime_r(&t, &tmbuf);
-	snprintf(date, 30,
+	snprintf(date, 29,
 		"%s, %02d %s %04d %02d:%02d:%02d GMT",
 		days[gmtime->tm_wday], gmtime->tm_mday,
 		months[gmtime->tm_mon], gmtime->tm_year + 1900,
@@ -1167,7 +1164,7 @@ PHP_HTTP_API STATUS _http_send_etag(const char *etag,
 
 	header_len = strlen("ETag: \"\"") + etag_len + 1;
 	etag_header = (char *) emalloc(header_len);
-	snprintf(etag_header, header_len, "ETag: \"%s\"", etag);
+	snprintf(etag_header, header_len - 1, "ETag: \"%s\"", etag);
 	ret = http_send_header(etag_header);
 	efree(etag_header);
 
@@ -1327,7 +1324,7 @@ PHP_HTTP_API http_range_status _http_get_request_ranges(zval *zranges,
 
 	if (strncmp(range, "bytes=", strlen("bytes="))) {
 		php_error_docref(NULL TSRMLS_CC, E_NOTICE, "Range header misses bytes=");
-		return RANGE_ERR;
+		return RANGE_NO;
 	}
 
 	ptr = &begin;
@@ -1433,7 +1430,7 @@ PHP_HTTP_API http_range_status _http_get_request_ranges(zval *zranges,
 			break;
 
 			default:
-				return RANGE_ERR;
+				return RANGE_NO;
 			break;
 		}
 	} while (c != 0);
@@ -1765,24 +1762,24 @@ PHP_HTTP_API STATUS _http_parse_headers(char *header, int header_len, zval *arra
 
 	while (header_len >= (line - begin)) {
 		int value_len = 0;
-		
+
 		switch (*line++)
 		{
 			case 0:
 				--value_len; /* we don't have CR so value length is one char less */
 			case '\n':
 				if (colon && ((!(*line - 1)) || ((*line != ' ') && (*line != '\t')))) {
-				
+
 					/* skip empty key */
 					if (header != colon) {
 						char *key = estrndup(header, colon - header);
 						value_len += line - colon - 1;
-							
+
 						/* skip leading ws */
 						while (isspace(*(++colon))) --value_len;
 						/* skip trailing ws */
 						while (isspace(colon[value_len - 1])) --value_len;
-						
+
 						if (value_len < 1) {
 							/* hm, empty header? */
 							add_assoc_stringl(array, key, "", 0, 0);
@@ -1791,7 +1788,7 @@ PHP_HTTP_API STATUS _http_parse_headers(char *header, int header_len, zval *arra
 						}
 						efree(key);
 					}
-					
+
 					colon = NULL;
 					value_len = 0;
 					header += line - header;
@@ -1813,7 +1810,7 @@ PHP_HTTP_API STATUS _http_parse_headers(char *header, int header_len, zval *arra
 PHP_HTTP_API void _http_get_request_headers(zval *array TSRMLS_DC)
 {
     char *key;
-    
+
     for (   zend_hash_internal_pointer_reset(HTTP_SERVER_VARS);
             zend_hash_get_current_key(HTTP_SERVER_VARS, &key, NULL, 0) != HASH_KEY_NON_EXISTANT;
             zend_hash_move_forward(HTTP_SERVER_VARS)) {