- update for release 0.17.0

[m6w6/ext-http] / http_message_object.c

diff --git a/http_message_object.c b/http_message_object.c
index 1db34f8e16cc19605e681290568e6df94e650fb3..921ba0ba459b6c8746e0b213110d2a33d71e8627 100644 (file)
--- a/http_message_object.c
+++ b/http_message_object.c
@@ -234,7 +234,12 @@ static zval *_http_message_object_read_prop(zval *object, zval *member, int type
        }
 #endif
 
-       return_value = &EG(uninitialized_zval);
+       if (type == BP_VAR_W) {
+               zend_error(E_ERROR, "Cannot access HttpMessage properties by reference or array key/index");
+               return NULL;
+       }
+       
+       ALLOC_ZVAL(return_value);
        return_value->refcount = 0;
        return_value->is_ref = 0;
 
@@ -245,34 +250,29 @@ static zval *_http_message_object_read_prop(zval *object, zval *member, int type
 #endif
        {
                case HTTP_MSG_PROPHASH_TYPE:
+               case HTTP_MSG_CHILD_PROPHASH_TYPE:
                        RETVAL_LONG(msg->type);
                break;
 
                case HTTP_MSG_PROPHASH_HTTP_VERSION:
+               case HTTP_MSG_CHILD_PROPHASH_HTTP_VERSION:
                        RETVAL_DOUBLE(msg->http.version);
                break;
 
                case HTTP_MSG_PROPHASH_BODY:
+               case HTTP_MSG_CHILD_PROPHASH_BODY:
                        phpstr_fix(PHPSTR(msg));
                        RETVAL_PHPSTR(PHPSTR(msg), 0, 1);
                break;
 
                case HTTP_MSG_PROPHASH_HEADERS:
-                       /* This is needed for situations like
-                        * $this->headers['foo'] = 'bar';
-                        */
-                       if (type == BP_VAR_W) {
-                               return_value->refcount = 1;
-                               return_value->is_ref = 1;
-                               Z_TYPE_P(return_value) = IS_ARRAY;
-                               Z_ARRVAL_P(return_value) = &msg->hdrs;
-                       } else {
-                               array_init(return_value);
-                               zend_hash_copy(Z_ARRVAL_P(return_value), &msg->hdrs, (copy_ctor_func_t) zval_add_ref, NULL, sizeof(zval *));
-                       }
+               case HTTP_MSG_CHILD_PROPHASH_HEADERS:
+                       array_init(return_value);
+                       zend_hash_copy(Z_ARRVAL_P(return_value), &msg->hdrs, (copy_ctor_func_t) zval_add_ref, NULL, sizeof(zval *));
                break;
 
                case HTTP_MSG_PROPHASH_PARENT_MESSAGE:
+               case HTTP_MSG_CHILD_PROPHASH_PARENT_MESSAGE:
                        if (msg->parent) {
                                RETVAL_OBJVAL(obj->parent);
                        } else {
@@ -281,6 +281,7 @@ static zval *_http_message_object_read_prop(zval *object, zval *member, int type
                break;
 
                case HTTP_MSG_PROPHASH_REQUEST_METHOD:
+               case HTTP_MSG_CHILD_PROPHASH_REQUEST_METHOD:
                        if (HTTP_MSG_TYPE(REQUEST, msg) && msg->http.info.request.method) {
                                RETVAL_STRING(msg->http.info.request.method, 1);
                        } else {
@@ -289,6 +290,7 @@ static zval *_http_message_object_read_prop(zval *object, zval *member, int type
                break;
 
                case HTTP_MSG_PROPHASH_REQUEST_URI:
+               case HTTP_MSG_CHILD_PROPHASH_REQUEST_URI:
                        if (HTTP_MSG_TYPE(REQUEST, msg) && msg->http.info.request.URI) {
                                RETVAL_STRING(msg->http.info.request.URI, 1);
                        } else {
@@ -297,6 +299,7 @@ static zval *_http_message_object_read_prop(zval *object, zval *member, int type
                break;
 
                case HTTP_MSG_PROPHASH_RESPONSE_CODE:
+               case HTTP_MSG_CHILD_PROPHASH_RESPONSE_CODE:
                        if (HTTP_MSG_TYPE(RESPONSE, msg)) {
                                RETVAL_LONG(msg->http.info.response.code);
                        } else {
@@ -305,6 +308,7 @@ static zval *_http_message_object_read_prop(zval *object, zval *member, int type
                break;
                
                case HTTP_MSG_PROPHASH_RESPONSE_STATUS:
+               case HTTP_MSG_CHILD_PROPHASH_RESPONSE_STATUS:
                        if (HTTP_MSG_TYPE(RESPONSE, msg) && msg->http.info.response.status) {
                                RETVAL_STRING(msg->http.info.response.status, 1);
                        } else {
@@ -346,28 +350,33 @@ static void _http_message_object_write_prop(zval *object, zval *member, zval *va
 #endif
        {
                case HTTP_MSG_PROPHASH_TYPE:
+               case HTTP_MSG_CHILD_PROPHASH_TYPE:
                        convert_to_long_ex(&value);
                        http_message_set_type(msg, Z_LVAL_P(value));
                break;
 
                case HTTP_MSG_PROPHASH_HTTP_VERSION:
+               case HTTP_MSG_CHILD_PROPHASH_HTTP_VERSION:
                        convert_to_double_ex(&value);
                        msg->http.version = Z_DVAL_P(value);
                break;
 
                case HTTP_MSG_PROPHASH_BODY:
+               case HTTP_MSG_CHILD_PROPHASH_BODY:
                        convert_to_string_ex(&value);
                        phpstr_dtor(PHPSTR(msg));
                        phpstr_from_string_ex(PHPSTR(msg), Z_STRVAL_P(value), Z_STRLEN_P(value));
                break;
 
                case HTTP_MSG_PROPHASH_HEADERS:
+               case HTTP_MSG_CHILD_PROPHASH_HEADERS:
                        convert_to_array_ex(&value);
                        zend_hash_clean(&msg->hdrs);
                        zend_hash_copy(&msg->hdrs, Z_ARRVAL_P(value), (copy_ctor_func_t) zval_add_ref, NULL, sizeof(zval *));
                break;
 
                case HTTP_MSG_PROPHASH_PARENT_MESSAGE:
+               case HTTP_MSG_CHILD_PROPHASH_PARENT_MESSAGE:
                        if (Z_TYPE_P(value) == IS_OBJECT && instanceof_function(Z_OBJCE_P(value), http_message_object_ce TSRMLS_CC)) {
                                if (msg->parent) {
                                        zval tmp;
@@ -380,6 +389,7 @@ static void _http_message_object_write_prop(zval *object, zval *member, zval *va
                break;
 
                case HTTP_MSG_PROPHASH_REQUEST_METHOD:
+               case HTTP_MSG_CHILD_PROPHASH_REQUEST_METHOD:
                        if (HTTP_MSG_TYPE(REQUEST, msg)) {
                                convert_to_string_ex(&value);
                                STR_SET(msg->http.info.request.method, estrndup(Z_STRVAL_P(value), Z_STRLEN_P(value)));
@@ -387,6 +397,7 @@ static void _http_message_object_write_prop(zval *object, zval *member, zval *va
                break;
 
                case HTTP_MSG_PROPHASH_REQUEST_URI:
+               case HTTP_MSG_CHILD_PROPHASH_REQUEST_URI:
                        if (HTTP_MSG_TYPE(REQUEST, msg)) {
                                convert_to_string_ex(&value);
                                STR_SET(msg->http.info.request.URI, estrndup(Z_STRVAL_P(value), Z_STRLEN_P(value)));
@@ -394,6 +405,7 @@ static void _http_message_object_write_prop(zval *object, zval *member, zval *va
                break;
 
                case HTTP_MSG_PROPHASH_RESPONSE_CODE:
+               case HTTP_MSG_CHILD_PROPHASH_RESPONSE_CODE:
                        if (HTTP_MSG_TYPE(RESPONSE, msg)) {
                                convert_to_long_ex(&value);
                                msg->http.info.response.code = Z_LVAL_P(value);
@@ -401,6 +413,7 @@ static void _http_message_object_write_prop(zval *object, zval *member, zval *va
                break;
                
                case HTTP_MSG_PROPHASH_RESPONSE_STATUS:
+               case HTTP_MSG_CHILD_PROPHASH_RESPONSE_STATUS:
                        if (HTTP_MSG_TYPE(RESPONSE, msg)) {
                                convert_to_string_ex(&value);
                                STR_SET(msg->http.info.response.status, estrndup(Z_STRVAL_P(value), Z_STRLEN_P(value)));