projects
/
m6w6
/
ext-http
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
|
github
raw
|
patch
|
inline
| side by side (parent:
aee4dea
)
fix warning, and place bet on the safe side
author
Michael Wallner
<mike@php.net>
Fri, 22 May 2015 07:29:42 +0000
(09:29 +0200)
committer
Michael Wallner
<mike@php.net>
Fri, 22 May 2015 07:29:42 +0000
(09:29 +0200)
php_http_message_parser.c
patch
|
blob
|
history
diff --git
a/php_http_message_parser.c
b/php_http_message_parser.c
index 3b55efbab7b295a9cd210fa57cb409769a3a97e6..fae16f1dd867f4f4648e9f478ac584dcacd5462e 100644
(file)
--- a/
php_http_message_parser.c
+++ b/
php_http_message_parser.c
@@
-61,19
+61,21
@@
php_http_message_parser_t *php_http_message_parser_init(php_http_message_parser_
php_http_message_parser_state_t php_http_message_parser_state_push(php_http_message_parser_t *parser, unsigned argc, ...)
{
php_http_message_parser_state_t php_http_message_parser_state_push(php_http_message_parser_t *parser, unsigned argc, ...)
{
- php_http_message_parser_state_t state;
+ php_http_message_parser_state_t state
= PHP_HTTP_MESSAGE_PARSER_STATE_FAILURE
;
va_list va_args;
unsigned i;
va_list va_args;
unsigned i;
- /* short circuit */
- ZEND_PTR_STACK_RESIZE_IF_NEEDED((&parser->stack), argc);
+ if (argc > 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_message_parser_state_t);
- zend_ptr_stack_push(&parser->stack, (void *) state);
+ va_start(va_args, argc);
+ for (i = 0; i < argc; ++i) {
+ state = va_arg(va_args, php_http_message_parser_state_t);
+ zend_ptr_stack_push(&parser->stack, (void *) state);
+ }
+ va_end(va_args);
}
}
- va_end(va_args);
return state;
}
return state;
}