X-Git-Url: https://git.m6w6.name/?a=blobdiff_plain;f=php_http_header_parser.c;h=e14b44333b1efe44b23b6f05fe2ba65f14bd3d11;hb=49c4716aa5a97b2ae94491e74f5ff5c13c0ddee3;hp=e0dea5babf3e5987b678756696e2cc46c9d04e44;hpb=22f2117f08acf0fcf84d70a54ef168ff1627fc69;p=m6w6%2Fext-http

diff --git a/php_http_header_parser.c b/php_http_header_parser.c
index e0dea5b..e14b443 100644
--- a/php_http_header_parser.c
+++ b/php_http_header_parser.c
@@ -6,7 +6,7 @@
     | modification, are permitted provided that the conditions mentioned |
     | in the accompanying LICENSE file are met.                          |
     +--------------------------------------------------------------------+
-    | Copyright (c) 2004-2011, Michael Wallner <mike@php.net>            |
+    | Copyright (c) 2004-2014, Michael Wallner <mike@php.net>            |
     +--------------------------------------------------------------------+
 */
 
@@ -26,66 +26,62 @@ static const php_http_header_parser_state_spec_t php_http_header_parser_states[]
 		{PHP_HTTP_HEADER_PARSER_STATE_DONE,			0}
 };
 
-
-PHP_HTTP_API php_http_header_parser_t *php_http_header_parser_init(php_http_header_parser_t *parser TSRMLS_DC)
+php_http_header_parser_t *php_http_header_parser_init(php_http_header_parser_t *parser)
 {
 	if (!parser) {
 		parser = emalloc(sizeof(*parser));
 	}
 	memset(parser, 0, sizeof(*parser));
 
-	TSRMLS_SET_CTX(parser->ts);
-
 	return parser;
 }
 
-
-PHP_HTTP_API php_http_header_parser_state_t php_http_header_parser_state_push(php_http_header_parser_t *parser, unsigned argc, ...)
+php_http_header_parser_state_t php_http_header_parser_state_push(php_http_header_parser_t *parser, unsigned argc, ...)
 {
 	va_list va_args;
 	unsigned i;
 	php_http_header_parser_state_t state = 0;
 
+	/* short circuit */
+	ZEND_PTR_STACK_RESIZE_IF_NEEDED((&parser->stack), argc);
+
 	va_start(va_args, argc);
 	for (i = 0; i < argc; ++i) {
 		state = va_arg(va_args, php_http_header_parser_state_t);
-		zend_stack_push(&parser->stack, &state, sizeof(state));
+		zend_ptr_stack_push(&parser->stack, (void *) state);
 	}
 	va_end(va_args);
 
 	return state;
 }
 
-PHP_HTTP_API php_http_header_parser_state_t php_http_header_parser_state_is(php_http_header_parser_t *parser)
+php_http_header_parser_state_t php_http_header_parser_state_is(php_http_header_parser_t *parser)
 {
-	php_http_header_parser_state_t *state;
-
-	if (SUCCESS == zend_stack_top(&parser->stack, (void *) &state)) {
-		return *state;
+	if (parser->stack.top) {
+		return (php_http_header_parser_state_t) parser->stack.elements[parser->stack.top - 1];
 	}
+
 	return PHP_HTTP_HEADER_PARSER_STATE_START;
 }
 
-PHP_HTTP_API php_http_header_parser_state_t php_http_header_parser_state_pop(php_http_header_parser_t *parser)
+php_http_header_parser_state_t php_http_header_parser_state_pop(php_http_header_parser_t *parser)
 {
-	php_http_header_parser_state_t state, *state_ptr;
-	if (SUCCESS == zend_stack_top(&parser->stack, (void *) &state_ptr)) {
-		state = *state_ptr;
-		zend_stack_del_top(&parser->stack);
-		return state;
+	if (parser->stack.top) {
+		return (php_http_header_parser_state_t) zend_ptr_stack_pop(&parser->stack);
 	}
+
 	return PHP_HTTP_HEADER_PARSER_STATE_START;
 }
 
-PHP_HTTP_API void php_http_header_parser_dtor(php_http_header_parser_t *parser)
+void php_http_header_parser_dtor(php_http_header_parser_t *parser)
 {
-	zend_stack_destroy(&parser->stack);
+	zend_ptr_stack_destroy(&parser->stack);
 	php_http_info_dtor(&parser->info);
-	STR_FREE(parser->_key.str);
-	STR_FREE(parser->_val.str);
+	PTR_FREE(parser->_key.str);
+	PTR_FREE(parser->_val.str);
 }
 
-PHP_HTTP_API void php_http_header_parser_free(php_http_header_parser_t **parser)
+void php_http_header_parser_free(php_http_header_parser_t **parser)
 {
 	if (*parser) {
 		php_http_header_parser_dtor(*parser);
@@ -94,14 +90,15 @@ PHP_HTTP_API void php_http_header_parser_free(php_http_header_parser_t **parser)
 	}
 }
 
-PHP_HTTP_API STATUS php_http_header_parser_parse(php_http_header_parser_t *parser, php_http_buffer_t *buffer, unsigned flags, HashTable *headers, php_http_info_callback_t callback_func, void *callback_arg)
+php_http_header_parser_state_t php_http_header_parser_parse(php_http_header_parser_t *parser, php_http_buffer_t *buffer, unsigned flags, HashTable *headers, php_http_info_callback_t callback_func, void *callback_arg)
 {
-	TSRMLS_FETCH_FROM_CTX(parser->ts);
-
 	while (buffer->used || !php_http_header_parser_states[php_http_header_parser_state_is(parser)].need_data) {
 #if 0
-		const char *state[] = {"START", "KEY", "VALUE", "HEADER_DONE", "DONE"};
-		fprintf(stderr, "#HP: %s (avail:%zu, num:%d)\n", php_http_header_parser_state_is(parser) < 0 ? "FAILURE" : state[php_http_header_parser_state_is(parser)], buffer->used, headers?zend_hash_num_elements(headers):0);
+		const char *state[] = {"START", "KEY", "VALUE", "VALUE_EX", "HEADER_DONE", "DONE"};
+		int num_headers = headers ? zend_hash_num_elements(headers) : 0;
+		fprintf(stderr, "#HP: (%d) %s (avail:%zu, num:%d)\n", php_http_header_parser_state_is(parser),
+				php_http_header_parser_state_is(parser) < 0 ? "FAILURE" : state[php_http_header_parser_state_is(parser)],
+						buffer->used, num_headers);
 		_dpf(0, buffer->data, buffer->used);
 #endif
 		switch (php_http_header_parser_state_pop(parser)) {
@@ -128,10 +125,10 @@ PHP_HTTP_API STATUS php_http_header_parser_parse(php_http_header_parser_t *parse
 					/* end of headers */
 					php_http_buffer_cut(buffer, 0, eol_len);
 					php_http_header_parser_state_push(parser, 1, PHP_HTTP_HEADER_PARSER_STATE_DONE);
-				} else if (php_http_info_parse(&parser->info, php_http_buffer_fix(buffer)->data TSRMLS_CC)) {
+				} else if (php_http_info_parse(&parser->info, php_http_buffer_fix(buffer)->data)) {
 					/* new message starting with request/response line */
 					if (callback_func) {
-						callback_func(callback_arg, &headers, &parser->info TSRMLS_CC);
+						callback_func(callback_arg, &headers, &parser->info);
 					}
 					php_http_info_dtor(&parser->info);
 					php_http_buffer_cut(buffer, 0, eol_str + eol_len - buffer->data);
@@ -155,7 +152,7 @@ PHP_HTTP_API STATUS php_http_header_parser_parse(php_http_header_parser_t *parse
 
 #define SET_ADD_VAL(slen, eol_len) \
 	do { \
-		char *ptr = buffer->data; \
+		const char *ptr = buffer->data; \
 		size_t len = slen; \
 		 \
 		while (len > 0 && PHP_HTTP_IS_CTYPE(space, *ptr)) { \
@@ -216,25 +213,25 @@ PHP_HTTP_API STATUS php_http_header_parser_parse(php_http_header_parser_t *parse
 
 			case PHP_HTTP_HEADER_PARSER_STATE_HEADER_DONE:
 				if (parser->_key.str && parser->_val.str) {
-					zval array, **exist;
+					zval tmp, *exist;
 
 					if (!headers && callback_func) {
-						callback_func(callback_arg, &headers, NULL TSRMLS_CC);
+						callback_func(callback_arg, &headers, NULL);
 					}
 
-					INIT_PZVAL_ARRAY(&array, headers);
 					php_http_pretty_key(parser->_key.str, parser->_key.len, 1, 1);
-					if (SUCCESS == zend_symtable_find(headers, parser->_key.str, parser->_key.len + 1, (void *) &exist)) {
-						convert_to_array(*exist);
-						add_next_index_stringl(*exist, parser->_val.str, parser->_val.len, 0);
+					if ((exist = zend_symtable_str_find(headers, parser->_key.str, parser->_key.len))) {
+						convert_to_array(exist);
+						add_next_index_str(exist, php_http_cs2zs(parser->_val.str, parser->_val.len));
 					} else {
-						add_assoc_stringl_ex(&array, parser->_key.str, parser->_key.len + 1, parser->_val.str, parser->_val.len, 0);
+						ZVAL_STR(&tmp, php_http_cs2zs(parser->_val.str, parser->_val.len));
+						zend_symtable_str_update(headers, parser->_key.str, parser->_key.len, &tmp);
 					}
 					parser->_val.str = NULL;
 				}
 
-				STR_SET(parser->_key.str, NULL);
-				STR_SET(parser->_val.str, NULL);
+				PTR_SET(parser->_key.str, NULL);
+				PTR_SET(parser->_val.str, NULL);
 
 				php_http_header_parser_state_push(parser, 1, PHP_HTTP_HEADER_PARSER_STATE_KEY);
 				break;