X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-http;a=blobdiff_plain;f=php_http_message.c;h=75d7fe5396e9e4ab41c9125a040bbb559d4e4c30;hp=ae7ca6e9706c16ac80e34c37fded29215329e956;hb=1571b39b2988b27c93cfa2cd9114439784bff2d6;hpb=be9585b914061f84c5d5939673d29ecc048ab1d7

diff --git a/php_http_message.c b/php_http_message.c
index ae7ca6e..75d7fe5 100644
--- a/php_http_message.c
+++ b/php_http_message.c
@@ -275,6 +275,8 @@ void php_http_message_update_headers(php_http_message_t *msg)
 
 	if (php_http_message_body_stream(msg->body)->readfilters.head) {
 		/* if a read stream filter is attached to the body the caller must also care for the headers */
+	} else if (php_http_message_header(msg, ZEND_STRL("Content-Range"))) {
+		/* don't mess around with a Content-Range message */
 	} else if ((size = php_http_message_body_size(msg->body))) {
 		ZVAL_LONG(&h, size);
 		zend_hash_str_update(&msg->hdrs, "Content-Length", lenof("Content-Length"), &h);
@@ -299,6 +301,7 @@ void php_http_message_update_headers(php_http_message_t *msg)
 		}
 	} else if ((cl = php_http_message_header_string(msg, ZEND_STRL("Content-Length")))) {
 		if (!zend_string_equals_literal(cl, "0")) {
+			/* body->size == 0, so get rid of old Content-Length */
 			zend_hash_str_del(&msg->hdrs, ZEND_STRL("Content-Length"));
 		}
 		zend_string_release(cl);
@@ -493,7 +496,6 @@ void php_http_message_free(php_http_message_t **message)
 
 static zval *php_http_message_object_read_prop(zval *object, zval *member, int type, void **cache_slot, zval *rv);
 static void php_http_message_object_write_prop(zval *object, zval *member, zval *value, void **cache_slot);
-static HashTable *php_http_message_object_get_props(zval *object);
 
 static zend_object_handlers php_http_message_object_handlers;
 static HashTable php_http_message_object_prophandlers;
@@ -687,7 +689,8 @@ void php_http_message_object_reverse(zval *zmsg, zval *return_value)
 
 		/* add ref, because we previously have not been a parent message */
 		Z_ADDREF_P(zmsg);
-		RETVAL_OBJECT(&objects[last]->zo, 1);
+		/* no addref, because we've been a parent message previously */
+		RETVAL_OBJECT(&objects[last]->zo, 0);
 
 		efree(objects);
 	} else {
@@ -894,7 +897,7 @@ static void php_http_message_object_write_prop(zval *object, zval *member, zval
 	zend_string_release(member_name);
 }
 
-static HashTable *php_http_message_object_get_props(zval *object)
+static HashTable *php_http_message_object_get_debug_info(zval *object, int *is_temp)
 {
 	zval tmp;
 	php_http_message_object_t *obj = PHP_HTTP_OBJ(NULL, object);
@@ -903,13 +906,14 @@ static HashTable *php_http_message_object_get_props(zval *object)
 	size_t ver_len, url_len = 0;
 
 	PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
+	*is_temp = 0;
 	
 #define UPDATE_PROP(name_str, action_with_tmp) \
 	do { \
 		zend_property_info *pi; \
 		if ((pi = zend_hash_str_find_ptr(&obj->zo.ce->properties_info, name_str, lenof(name_str)))) { \
 			action_with_tmp; \
-			zend_hash_update(props, pi->name, &tmp); \
+			zend_hash_update_ind(props, pi->name, &tmp); \
 		} \
 	} while(0)
 
@@ -1106,7 +1110,7 @@ static PHP_METHOD(HttpMessage, getHeader)
 
 		if ((header = php_http_message_header(obj->message, header_str, header_len))) {
 			if (!header_ce) {
-				RETURN_ZVAL(header, 1, 1);
+				RETURN_ZVAL_FAST(header);
 			} else if (instanceof_function(header_ce, php_http_header_class_entry)) {
 				php_http_object_method_t cb;
 				zval argv[2];
@@ -1968,7 +1972,7 @@ PHP_MINIT_FUNCTION(http_message)
 	php_http_message_object_handlers.free_obj = php_http_message_object_free;
 	php_http_message_object_handlers.read_property = php_http_message_object_read_prop;
 	php_http_message_object_handlers.write_property = php_http_message_object_write_prop;
-	php_http_message_object_handlers.get_properties = php_http_message_object_get_props;
+	php_http_message_object_handlers.get_debug_info = php_http_message_object_get_debug_info;
 	php_http_message_object_handlers.get_property_ptr_ptr = NULL;
 
 	zend_class_implements(php_http_message_class_entry, 3, spl_ce_Countable, zend_ce_serializable, zend_ce_iterator);