mirror of
https://github.com/php/php-src.git
synced 2025-08-16 14:08:47 +02:00
- do not use 64bit integer instead of size_t (can't be alloc'ed), or when the actual possible values are 32bit or lower only
This commit is contained in:
parent
8e0c2e1cb9
commit
0fd3f882cd
2 changed files with 6 additions and 6 deletions
|
@ -323,12 +323,12 @@ size_t php_http_parser_execute (php_http_parser *parser,
|
||||||
{
|
{
|
||||||
char c, ch;
|
char c, ch;
|
||||||
const char *p = data, *pe;
|
const char *p = data, *pe;
|
||||||
int64_t to_read;
|
size_t to_read;
|
||||||
|
|
||||||
enum state state = (enum state) parser->state;
|
enum state state = (enum state) parser->state;
|
||||||
enum header_states header_state = (enum header_states) parser->header_state;
|
enum header_states header_state = (enum header_states) parser->header_state;
|
||||||
uint64_t index = parser->index;
|
uint32_t index = parser->index;
|
||||||
uint64_t nread = parser->nread;
|
uint32_t nread = parser->nread;
|
||||||
|
|
||||||
/* technically we could combine all of these (except for url_mark) into one
|
/* 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
|
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:
|
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 (to_read > 0) {
|
||||||
if (settings->on_body) settings->on_body(parser, p, to_read);
|
if (settings->on_body) settings->on_body(parser, p, to_read);
|
||||||
p += to_read - 1;
|
p += to_read - 1;
|
||||||
|
@ -1510,7 +1510,7 @@ size_t php_http_parser_execute (php_http_parser *parser,
|
||||||
{
|
{
|
||||||
assert(parser->flags & F_CHUNKED);
|
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 (to_read > 0) {
|
||||||
if (settings->on_body) settings->on_body(parser, p, to_read);
|
if (settings->on_body) settings->on_body(parser, p, to_read);
|
||||||
|
|
|
@ -112,7 +112,7 @@ struct php_http_parser {
|
||||||
unsigned char index;
|
unsigned char index;
|
||||||
|
|
||||||
uint32_t nread;
|
uint32_t nread;
|
||||||
int64_t content_length;
|
size_t content_length;
|
||||||
|
|
||||||
/** READ-ONLY **/
|
/** READ-ONLY **/
|
||||||
unsigned short http_major;
|
unsigned short http_major;
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue