X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-http;a=blobdiff_plain;f=src%2Fphp_http_message_body.c;h=ad8a2b12f7827c80ffac690ae392d4c3ccc67ebc;hp=518dbc9b71ef34b57362a63463a2f100c7d5e3b7;hb=e44e3ceb60817cfdd17945a3f6043816e134aa75;hpb=a81472d0108d16e680980a9e0c1b4af0d629c43a diff --git a/src/php_http_message_body.c b/src/php_http_message_body.c index 518dbc9..ad8a2b1 100644 --- a/src/php_http_message_body.c +++ b/src/php_http_message_body.c @@ -214,7 +214,7 @@ size_t php_http_message_body_append(php_http_message_body_t *body, const char *b written = php_stream_write(s, buf, len); if (written != len) { - php_error_docref(NULL, E_WARNING, "Failed to append %zu bytes to body; wrote %zu", len, written); + php_error_docref(NULL, E_WARNING, "Failed to append %zu bytes to body; wrote %zu", len, written == (size_t) -1 ? 0 : written); } return len; @@ -330,7 +330,7 @@ static inline char *format_key(php_http_arrkey_t *key, const char *prefix) { } else if (key->key) { new_key = estrdup(key->key->val); } else { - new_key = estrdup(""); + spprintf(&new_key, 0, "%lu", key->h); } return new_key; @@ -392,14 +392,17 @@ static ZEND_RESULT_CODE add_recursive_files(php_http_message_body_t *body, const ZEND_HASH_FOREACH_KEY_VAL_IND(files, key.h, key.key, val) { if (Z_TYPE_P(val) == IS_ARRAY || Z_TYPE_P(val) == IS_OBJECT) { - char *str = format_key(&key, name); + char *str = key.key ? format_key(&key, name) : NULL; + const char *prefix = str ? str : name; - if (SUCCESS != add_recursive_files(body, str, HASH_OF(val))) { + if (SUCCESS != add_recursive_files(body, prefix, HASH_OF(val))) { efree(str); HT_UNPROTECT_RECURSION(files); return FAILURE; } - efree(str); + if (str) { + efree(str); + } } } ZEND_HASH_FOREACH_END(); @@ -429,7 +432,7 @@ static ZEND_RESULT_CODE add_recursive_files(php_http_message_body_t *body, const return FAILURE; } else { zend_string *znc = zval_get_string(zname), *ztc = zval_get_string(ztype); - php_http_arrkey_t arrkey = {0, znc}; + php_http_arrkey_t arrkey = {0, znc, 0, 0}; char *key = format_key(&arrkey, name); ZEND_RESULT_CODE ret = php_http_message_body_add_form_file(body, key, ztc->val, zfc->val, stream); @@ -587,10 +590,10 @@ php_http_message_body_object_t *php_http_message_body_object_new_ex(zend_class_e return o; } -zend_object *php_http_message_body_object_clone(zval *object) +zend_object *php_http_message_body_object_clone(zend_object *object) { php_http_message_body_object_t *new_obj; - php_http_message_body_object_t *old_obj = PHP_HTTP_OBJ(NULL, object); + php_http_message_body_object_t *old_obj = PHP_HTTP_OBJ(object, NULL); php_http_message_body_t *body = php_http_message_body_copy(old_obj->body, NULL); new_obj = php_http_message_body_object_new_ex(old_obj->zo.ce, body); @@ -599,10 +602,10 @@ zend_object *php_http_message_body_object_clone(zval *object) return &new_obj->zo; } -static HashTable *php_http_message_body_object_get_gc(zval *object, zval **table, int *n) +static HashTable *php_http_message_body_object_get_gc(zend_object *object, zval **table, int *n) { - php_http_message_body_object_t *obj = PHP_HTTP_OBJ(NULL, object); - HashTable *props = Z_OBJPROP_P(object); + php_http_message_body_object_t *obj = PHP_HTTP_OBJ(object, NULL); + HashTable *props = object->handlers->get_properties(object); uint32_t count = zend_hash_num_elements(props); obj->gc = erealloc(obj->gc, (1 + count) * sizeof(zval)); @@ -907,7 +910,7 @@ PHP_METHOD(HttpMessageBody, stat) } static zend_function_entry php_http_message_body_methods[] = { - PHP_ME(HttpMessageBody, __construct, ai_HttpMessageBody___construct, ZEND_ACC_PUBLIC|ZEND_ACC_CTOR) + PHP_ME(HttpMessageBody, __construct, ai_HttpMessageBody___construct, ZEND_ACC_PUBLIC) PHP_ME(HttpMessageBody, __toString, ai_HttpMessageBody___toString, ZEND_ACC_PUBLIC) PHP_MALIAS(HttpMessageBody, toString, __toString, ai_HttpMessageBody___toString, ZEND_ACC_PUBLIC) PHP_MALIAS(HttpMessageBody, serialize, __toString, ai_HttpMessageBody___toString, ZEND_ACC_PUBLIC)