- do not use 64bit integer instead of size_t (can't be alloc'ed), or when the actual...

diff --git a/sapi/cli/php_http_parser.c b/sapi/cli/php_http_parser.c
index 5d5e44e3e551fde5a6de1a5e97d3741044ee9b7c..e58fba3410f03cd2538b8999e792a6d0a8765f87 100644 (file)
--- a/sapi/cli/php_http_parser.c
+++ b/sapi/cli/php_http_parser.c
@@ -323,12 +323,12 @@ size_t php_http_parser_execute (php_http_parser *parser,
 {
   char c, ch;
   const char *p = data, *pe;
-  int64_t to_read;
+  size_t to_read;
 
   enum state state = (enum state) parser->state;
   enum header_states header_state = (enum header_states) parser->header_state;
-  uint64_t index = parser->index;
-  uint64_t nread = parser->nread;
+  uint32_t index = parser->index;
+  uint32_t nread = parser->nread;
 
   /* technically we could combine all of these (except for url_mark) into one
      variable, saving stack space, but it seems more clear to have them
@@ -1425,7 +1425,7 @@ size_t php_http_parser_execute (php_http_parser *parser,
       }
 
       case s_body_identity:
-        to_read = MIN(pe - p, (int64_t)parser->content_length);
+        to_read = MIN(pe - p, (size_t)parser->content_length);
         if (to_read > 0) {
           if (settings->on_body) settings->on_body(parser, p, to_read);
           p += to_read - 1;
@@ -1510,7 +1510,7 @@ size_t php_http_parser_execute (php_http_parser *parser,
       {
         assert(parser->flags & F_CHUNKED);
 
-        to_read = MIN(pe - p, (int64_t)(parser->content_length));
+        to_read = MIN(pe - p, (size_t)(parser->content_length));
 
         if (to_read > 0) {
           if (settings->on_body) settings->on_body(parser, p, to_read);

diff --git a/sapi/cli/php_http_parser.h b/sapi/cli/php_http_parser.h
index 353550f0171d55612ed11d797b9b1942aaa112e4..8be44739a2bc4bdaa440b6e39c6f8b613cc7c3e2 100644 (file)
--- a/sapi/cli/php_http_parser.h
+++ b/sapi/cli/php_http_parser.h
@@ -112,7 +112,7 @@ struct php_http_parser {
   unsigned char index;
 
   uint32_t nread;
-  int64_t content_length;
+  size_t content_length;
 
   /** READ-ONLY **/
   unsigned short http_major;