X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-http;a=blobdiff_plain;f=php_http_message.c;h=3ac96550b5ac20bf8d819775c635d879b034ed66;hp=c0e60f29be326642eb49bae683a71edfcec3d641;hb=f05a032e5e994a8b1ea8b5ab0291f1d1d6d42355;hpb=87702149e0c422104aec0ddb7391f91392be0376

diff --git a/php_http_message.c b/php_http_message.c
index c0e60f2..3ac9655 100644
--- a/php_http_message.c
+++ b/php_http_message.c
@@ -133,7 +133,7 @@ PHP_HTTP_API php_http_message_t *php_http_message_parse(php_http_message_t *msg,
 	if (greedy) {
 		flags |= PHP_HTTP_MESSAGE_PARSER_GREEDY;
 	}
-	if (FAILURE == php_http_message_parser_parse(&p, &buf, flags, &msg)) {
+	if (PHP_HTTP_MESSAGE_PARSER_STATE_FAILURE == php_http_message_parser_parse(&p, &buf, flags, &msg)) {
 		if (free_msg) {
 			php_http_message_free(&msg);
 		}
@@ -689,11 +689,9 @@ static STATUS php_http_message_object_add_prophandler(const char *prop_str, size
 	php_http_message_object_prophandler_t h = { read, write };
 	return zend_hash_add(&php_http_message_object_prophandlers, prop_str, prop_len + 1, (void *) &h, sizeof(h), NULL);
 }
-/*
 static int php_http_message_object_has_prophandler(const char *prop_str, size_t prop_len) {
 	return zend_hash_exists(&php_http_message_object_prophandlers, prop_str, prop_len + 1);
 }
-*/
 static STATUS php_http_message_object_get_prophandler(const char *prop_str, size_t prop_len, php_http_message_object_prophandler_t **handler) {
 	return zend_hash_find(&php_http_message_object_prophandlers, prop_str, prop_len + 1, (void *) handler);
 }
@@ -862,11 +860,21 @@ PHP_MSHUTDOWN_FUNCTION(http_message)
 	return SUCCESS;
 }
 
+#define PHP_HTTP_MESSAGE_OBJECT_INIT(obj) \
+	do { \
+		if (!obj->message) { \
+			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC); \
+		} \
+	} while(0)
+
+
 void php_http_message_object_reverse(zval *this_ptr, zval *return_value TSRMLS_DC)
 {
 	int i = 0;
 	php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
+	PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
+
 	/* count */
 	php_http_message_count(i, obj->message);
 
@@ -1004,6 +1012,9 @@ STATUS php_http_message_object_set_body(php_http_message_object_t *msg_obj, zval
 	msg_obj->message->body = php_http_message_body_init(&body_obj->body, NULL TSRMLS_CC);
 	msg_obj->body = Z_OBJVAL_P(zbody);
 
+	if (tmp) {
+		FREE_ZVAL(tmp);
+	}
 	return SUCCESS;
 }
 
@@ -1078,10 +1089,9 @@ void php_http_message_object_free(void *object TSRMLS_DC)
 
 static zval **php_http_message_object_get_prop_ptr(zval *object, zval *member PHP_HTTP_ZEND_LITERAL_DC TSRMLS_DC)
 {
-	php_http_message_object_prophandler_t *handler;
 	zval *copy = php_http_ztyp(IS_STRING, member);
 
-	if (SUCCESS == php_http_message_object_get_prophandler(Z_STRVAL_P(copy), Z_STRLEN_P(copy), &handler)) {
+	if (php_http_message_object_has_prophandler(Z_STRVAL_P(copy), Z_STRLEN_P(copy))) {
 		zval_ptr_dtor(&copy);
 		return &php_http_property_proxy_init(NULL, object, member, NULL TSRMLS_CC)->myself;
 	}
@@ -1096,6 +1106,8 @@ static zval *php_http_message_object_read_prop(zval *object, zval *member, int t
 	php_http_message_object_prophandler_t *handler;
 	zval *return_value, *copy = php_http_ztyp(IS_STRING, member);
 
+	PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
+
 	if (SUCCESS == php_http_message_object_get_prophandler(Z_STRVAL_P(copy), Z_STRLEN_P(copy), &handler)) {
 		if (type == BP_VAR_R) {
 			ALLOC_ZVAL(return_value);
@@ -1122,6 +1134,8 @@ static void php_http_message_object_write_prop(zval *object, zval *member, zval
 	php_http_message_object_prophandler_t *handler;
 	zval *copy = php_http_ztyp(IS_STRING, member);
 
+	PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
+
 	if (SUCCESS == php_http_message_object_get_prophandler(Z_STRVAL_P(copy), Z_STRLEN_P(copy), &handler)) {
 		handler->write(obj, value TSRMLS_CC);
 	} else {
@@ -1131,7 +1145,6 @@ static void php_http_message_object_write_prop(zval *object, zval *member, zval
 	zval_ptr_dtor(&copy);
 }
 
-
 static HashTable *php_http_message_object_get_props(zval *object TSRMLS_DC)
 {
 	zval *headers;
@@ -1141,65 +1154,66 @@ static HashTable *php_http_message_object_get_props(zval *object TSRMLS_DC)
 	zval array, *parent, *body;
 	char *version;
 
+	PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
+	
 	INIT_PZVAL_ARRAY(&array, props);
-
-#define ASSOC_PROP(array, ptype, name, val) \
-	{ \
-		char *m_prop_name; \
-		int m_prop_len; \
-		zend_mangle_property_name(&m_prop_name, &m_prop_len, "*", 1, name, lenof(name), 0); \
-		add_assoc_ ##ptype## _ex(&array, m_prop_name, sizeof(name)+3, val); \
-		efree(m_prop_name); \
-	}
-#define ASSOC_STRING(array, name, val) ASSOC_STRINGL(array, name, val, strlen(val))
-#define ASSOC_STRINGL(array, name, val, len) ASSOC_STRINGL_EX(array, name, val, len, 1)
-#define ASSOC_STRINGL_EX(array, name, val, len, cpy) \
-	{ \
-		char *m_prop_name; \
-		int m_prop_len; \
-		zend_mangle_property_name(&m_prop_name, &m_prop_len, "*", 1, name, lenof(name), 0); \
-		add_assoc_stringl_ex(&array, m_prop_name, sizeof(name)+3, val, len, cpy); \
-		efree(m_prop_name); \
-	}
-
-	ASSOC_PROP(array, long, "type", msg->type);
-	ASSOC_STRINGL_EX(array, "httpVersion", version, spprintf(&version, 0, "%u.%u", msg->http.version.major, msg->http.version.minor), 0);
+	
+#define ASSOC_PROP(ptype, n, val) \
+	do { \
+		zend_property_info *pi; \
+		if (SUCCESS == zend_hash_find(&obj->zo.ce->properties_info, n, sizeof(n), (void *) &pi)) { \
+			add_assoc_ ##ptype## _ex(&array, pi->name, pi->name_length + 1, val); \
+		} \
+	} while(0) \
+
+#define ASSOC_STRING(name, val) ASSOC_STRINGL(name, val, strlen(val))
+#define ASSOC_STRINGL(name, val, len) ASSOC_STRINGL_EX(name, val, len, 1)
+#define ASSOC_STRINGL_EX(n, val, len, cpy) \
+	do { \
+		zend_property_info *pi; \
+		if (SUCCESS == zend_hash_find(&obj->zo.ce->properties_info, n, sizeof(n), (void *) &pi)) { \
+			add_assoc_stringl_ex(&array, pi->name, pi->name_length + 1, val, len, cpy); \
+		} \
+	} while(0)
+
+	ASSOC_PROP(long, "type", msg->type);
+	ASSOC_STRINGL_EX("httpVersion", version, spprintf(&version, 0, "%u.%u", msg->http.version.major, msg->http.version.minor), 0);
 
 	switch (msg->type) {
 		case PHP_HTTP_REQUEST:
-			ASSOC_PROP(array, long, "responseCode", 0);
-			ASSOC_STRINGL(array, "responseStatus", "", 0);
-			ASSOC_STRING(array, "requestMethod", STR_PTR(msg->http.info.request.method));
-			ASSOC_STRING(array, "requestUrl", STR_PTR(msg->http.info.request.url));
+			ASSOC_PROP(long, "responseCode", 0);
+			ASSOC_STRINGL("responseStatus", "", 0);
+			ASSOC_STRING("requestMethod", STR_PTR(msg->http.info.request.method));
+			ASSOC_STRING("requestUrl", STR_PTR(msg->http.info.request.url));
 			break;
 
 		case PHP_HTTP_RESPONSE:
-			ASSOC_PROP(array, long, "responseCode", msg->http.info.response.code);
-			ASSOC_STRING(array, "responseStatus", STR_PTR(msg->http.info.response.status));
-			ASSOC_STRINGL(array, "requestMethod", "", 0);
-			ASSOC_STRINGL(array, "requestUrl", "", 0);
+			ASSOC_PROP(long, "responseCode", msg->http.info.response.code);
+			ASSOC_STRING("responseStatus", STR_PTR(msg->http.info.response.status));
+			ASSOC_STRINGL("requestMethod", "", 0);
+			ASSOC_STRINGL("requestUrl", "", 0);
 			break;
 
 		case PHP_HTTP_NONE:
 		default:
-			ASSOC_PROP(array, long, "responseCode", 0);
-			ASSOC_STRINGL(array, "responseStatus", "", 0);
-			ASSOC_STRINGL(array, "requestMethod", "", 0);
-			ASSOC_STRINGL(array, "requestUrl", "", 0);
+			ASSOC_PROP(long, "responseCode", 0);
+			ASSOC_STRINGL("responseStatus", "", 0);
+			ASSOC_STRINGL("requestMethod", "", 0);
+			ASSOC_STRINGL("requestUrl", "", 0);
 			break;
 	}
 
 	MAKE_STD_ZVAL(headers);
 	array_init(headers);
 	zend_hash_copy(Z_ARRVAL_P(headers), &msg->hdrs, (copy_ctor_func_t) zval_add_ref, NULL, sizeof(zval *));
-	ASSOC_PROP(array, zval, "headers", headers);
+	ASSOC_PROP(zval, "headers", headers);
 
 	MAKE_STD_ZVAL(body);
 	if (!obj->body.handle) {
 		php_http_new(&obj->body, php_http_message_body_get_class_entry(), (php_http_new_t) php_http_message_body_object_new_ex, NULL, (void *) php_http_message_body_init(&obj->message->body, NULL TSRMLS_CC), NULL TSRMLS_CC);
 	}
 	ZVAL_OBJVAL(body, obj->body, 1);
-	ASSOC_PROP(array, zval, "body", body);
+	ASSOC_PROP(zval, "body", body);
 
 	MAKE_STD_ZVAL(parent);
 	if (msg->parent) {
@@ -1207,7 +1221,7 @@ static HashTable *php_http_message_object_get_props(zval *object TSRMLS_DC)
 	} else {
 		ZVAL_NULL(parent);
 	}
-	ASSOC_PROP(array, zval, "parentMessage", parent);
+	ASSOC_PROP(zval, "parentMessage", parent);
 
 	return props;
 }
@@ -1254,9 +1268,7 @@ PHP_METHOD(HttpMessage, __construct)
 				}
 			}
 		}
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 	} end_error_handling();
 
 }
@@ -1267,9 +1279,8 @@ PHP_METHOD(HttpMessage, getBody)
 		if (SUCCESS == zend_parse_parameters_none()) {
 			php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-			if (!obj->message) {
-				obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-			}
+			PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
+
 			if (!obj->body.handle) {
 				php_http_message_body_addref(obj->message->body);
 				php_http_new(&obj->body, php_http_message_body_get_class_entry(), (php_http_new_t) php_http_message_body_object_new_ex, NULL, obj->message->body, NULL TSRMLS_CC);
@@ -1288,9 +1299,7 @@ PHP_METHOD(HttpMessage, setBody)
 	if (SUCCESS == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "O", &zbody, php_http_message_body_get_class_entry())) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 		php_http_message_object_prophandler_set_body(obj, zbody TSRMLS_CC);
 	}
 	RETVAL_ZVAL(getThis(), 1, 0);
@@ -1304,6 +1313,7 @@ PHP_METHOD(HttpMessage, addBody)
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 		php_http_message_body_object_t *new_obj = zend_object_store_get_object(new_body TSRMLS_CC);
 
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 		php_http_message_body_to_callback(new_obj->body, (php_http_pass_callback_t) php_http_message_body_append, obj->message->body, 0, 0);
 	}
 	RETVAL_ZVAL(getThis(), 1, 0);
@@ -1319,9 +1329,7 @@ PHP_METHOD(HttpMessage, getHeader)
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 		zval *header;
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		if ((header = php_http_message_header(obj->message, header_str, header_len, 0))) {
 			RETURN_ZVAL(header, 1, 1);
@@ -1335,9 +1343,7 @@ PHP_METHOD(HttpMessage, getHeaders)
 	if (SUCCESS == zend_parse_parameters_none()) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		array_init(return_value);
 		array_copy(&obj->message->hdrs, Z_ARRVAL_P(return_value));
@@ -1354,9 +1360,7 @@ PHP_METHOD(HttpMessage, setHeader)
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 		char *name = php_http_pretty_key(estrndup(name_str, name_len), name_len, 1, 1);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		if (!zvalue) {
 			zend_symtable_del(&obj->message->hdrs, name, name_len + 1);
@@ -1376,9 +1380,7 @@ PHP_METHOD(HttpMessage, setHeaders)
 	if (SUCCESS == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a/!", &new_headers)) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		zend_hash_clean(&obj->message->hdrs);
 		if (new_headers) {
@@ -1399,9 +1401,7 @@ PHP_METHOD(HttpMessage, addHeader)
 		char *name = php_http_pretty_key(estrndup(name_str, name_len), name_len, 1, 1);
 		zval *header;
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		Z_ADDREF_P(zvalue);
 		if ((header = php_http_message_header(obj->message, name, name_len, 0))) {
@@ -1424,9 +1424,7 @@ PHP_METHOD(HttpMessage, addHeaders)
 	if (SUCCESS == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a|b", &new_headers, &append)) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		array_join(Z_ARRVAL_P(new_headers), &obj->message->hdrs, append, ARRAY_JOIN_STRONLY|ARRAY_JOIN_PRETTIFY);
 	}
@@ -1438,9 +1436,7 @@ PHP_METHOD(HttpMessage, getType)
 	if (SUCCESS == zend_parse_parameters_none()) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		RETURN_LONG(obj->message->type);
 	}
@@ -1453,9 +1449,7 @@ PHP_METHOD(HttpMessage, setType)
 	if (SUCCESS == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &type)) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		php_http_message_set_type(obj->message, type);
 	}
@@ -1467,9 +1461,7 @@ PHP_METHOD(HttpMessage, getInfo)
 	if (SUCCESS == zend_parse_parameters_none()) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		switch (obj->message->type) {
 			case PHP_HTTP_REQUEST:
@@ -1498,9 +1490,7 @@ PHP_METHOD(HttpMessage, setInfo)
 	&&		php_http_info_parse(&inf, str TSRMLS_CC)) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		php_http_message_set_info(obj->message, &inf);
 		php_http_info_dtor(&inf);
@@ -1515,9 +1505,7 @@ PHP_METHOD(HttpMessage, getHttpVersion)
 		size_t len;
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		php_http_version_to_string(&obj->message->http.version, &str, &len, NULL, NULL TSRMLS_CC);
 		RETURN_STRINGL(str, len, 0);
@@ -1535,9 +1523,7 @@ PHP_METHOD(HttpMessage, setHttpVersion)
 		php_http_version_t version;
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		if (php_http_version_parse(&version, v_str TSRMLS_CC)) {
 			obj->message->http.version = version;
@@ -1551,9 +1537,7 @@ PHP_METHOD(HttpMessage, getResponseCode)
 	if (SUCCESS == zend_parse_parameters_none()) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		PHP_HTTP_MESSAGE_TYPE_CHECK(RESPONSE, obj->message, RETURN_FALSE);
 		RETURN_LONG(obj->message->http.info.response.code);
@@ -1569,9 +1553,7 @@ PHP_METHOD(HttpMessage, setResponseCode)
 	if (SUCCESS == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l|b", &code, &strict)) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		PHP_HTTP_MESSAGE_TYPE_CHECK(RESPONSE, obj->message, RETURN_FALSE);
 		if (strict && (code < 100 || code > 599)) {
@@ -1590,9 +1572,7 @@ PHP_METHOD(HttpMessage, getResponseStatus)
 	if (SUCCESS == zend_parse_parameters_none()) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		PHP_HTTP_MESSAGE_TYPE_CHECK(RESPONSE, obj->message, RETURN_FALSE);
 		if (obj->message->http.info.response.status) {
@@ -1613,9 +1593,7 @@ PHP_METHOD(HttpMessage, setResponseStatus)
 	if (SUCCESS == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &status, &status_len)) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		PHP_HTTP_MESSAGE_TYPE_CHECK(RESPONSE, obj->message, RETURN_FALSE);
 		STR_SET(obj->message->http.info.response.status, estrndup(status, status_len));
@@ -1628,9 +1606,7 @@ PHP_METHOD(HttpMessage, getRequestMethod)
 	if (SUCCESS == zend_parse_parameters_none()) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		PHP_HTTP_MESSAGE_TYPE_CHECK(REQUEST, obj->message, RETURN_FALSE);
 		if (obj->message->http.info.request.method) {
@@ -1651,9 +1627,7 @@ PHP_METHOD(HttpMessage, setRequestMethod)
 	if (SUCCESS == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &method, &method_len)) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		PHP_HTTP_MESSAGE_TYPE_CHECK(REQUEST, obj->message, RETURN_FALSE);
 		if (method_len < 1) {
@@ -1671,9 +1645,7 @@ PHP_METHOD(HttpMessage, getRequestUrl)
 	if (SUCCESS == zend_parse_parameters_none()) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		PHP_HTTP_MESSAGE_TYPE_CHECK(REQUEST, obj->message, RETURN_FALSE);
 		if (obj->message->http.info.request.url) {
@@ -1694,9 +1666,7 @@ PHP_METHOD(HttpMessage, setRequestUrl)
 	if (SUCCESS == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &url_str, &url_len)) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		PHP_HTTP_MESSAGE_TYPE_CHECK(REQUEST, obj->message, RETURN_FALSE);
 		if (url_len < 1) {
@@ -1715,9 +1685,7 @@ PHP_METHOD(HttpMessage, getParentMessage)
 		if (SUCCESS == zend_parse_parameters_none()) {
 			php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-			if (!obj->message) {
-				obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-			}
+			PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 			if (obj->message->parent) {
 				RETVAL_OBJVAL(obj->parent, 1);
@@ -1737,9 +1705,7 @@ PHP_METHOD(HttpMessage, toString)
 		char *string;
 		size_t length;
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		if (include_parent) {
 			php_http_message_serialize(obj->message, &string, &length);
@@ -1761,12 +1727,9 @@ PHP_METHOD(HttpMessage, toStream)
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 		php_stream *s;
 
-		php_stream_from_zval(s, &zstream);
-
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
+		php_stream_from_zval(s, &zstream);
 		php_http_message_to_callback(obj->message, (php_http_pass_callback_t) _php_stream_write, s);
 	}
 }
@@ -1778,6 +1741,8 @@ PHP_METHOD(HttpMessage, toCallback)
 	if (SUCCESS == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "f", &fcd.fci, &fcd.fcc)) {
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
+
 		fcd.fcz = getThis();
 		Z_ADDREF_P(fcd.fcz);
 		TSRMLS_SET_CTX(fcd.ts);
@@ -1798,9 +1763,7 @@ PHP_METHOD(HttpMessage, serialize)
 		char *string;
 		size_t length;
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		php_http_message_serialize(obj->message, &string, &length);
 		RETURN_STRINGL(string, length, 0);
@@ -1836,9 +1799,7 @@ PHP_METHOD(HttpMessage, detach)
 		if (SUCCESS == zend_parse_parameters_none()) {
 			php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-			if (!obj->message) {
-				obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-			}
+			PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 			RETVAL_OBJVAL(php_http_message_object_new_ex(obj->zo.ce, php_http_message_copy_ex(obj->message, NULL, 0), NULL TSRMLS_CC), 0);
 		}
@@ -1855,9 +1816,8 @@ PHP_METHOD(HttpMessage, prepend)
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 		php_http_message_object_t *prepend_obj = zend_object_store_get_object(prepend TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
+
 		if (!prepend_obj->message) {
 			prepend_obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
 		}
@@ -1891,6 +1851,8 @@ PHP_METHOD(HttpMessage, isMultipart)
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 		char *boundary = NULL;
 
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
+
 		RETVAL_BOOL(php_http_message_is_multipart(obj->message, zboundary ? &boundary : NULL));
 
 		if (zboundary && boundary) {
@@ -1906,6 +1868,8 @@ PHP_METHOD(HttpMessage, splitMultipartBody)
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 		char *boundary = NULL;
 
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
+
 		if (php_http_message_is_multipart(obj->message, &boundary)) {
 			php_http_message_t *msg;
 
@@ -1923,9 +1887,7 @@ PHP_METHOD(HttpMessage, count)
 		long i = 0;
 		php_http_message_object_t *obj = zend_object_store_get_object(getThis() TSRMLS_CC);
 
-		if (!obj->message) {
-			obj->message = php_http_message_init(NULL, 0, NULL TSRMLS_CC);
-		}
+		PHP_HTTP_MESSAGE_OBJECT_INIT(obj);
 
 		php_http_message_count(i, obj->message);
 		RETURN_LONG(i);