HTTP/3: rearranged length check when parsing header.

The new code looks simpler and is similar to other checks.
This commit is contained in:
Roman Arutyunyan 2020-09-16 20:21:03 +01:00
parent ebbcc329cb
commit be719bbec8

View File

@ -212,6 +212,10 @@ ngx_http_v3_parse_headers(ngx_connection_t *c, ngx_http_v3_parse_headers_t *st,
break;
}
if (st->length == 0) {
return NGX_HTTP_V3_ERR_FRAME_ERROR;
}
st->state = sw_prefix;
break;
@ -225,7 +229,7 @@ ngx_http_v3_parse_headers(ngx_connection_t *c, ngx_http_v3_parse_headers_t *st,
case sw_prefix:
if (st->length-- == 0) {
if (--st->length == 0) {
return NGX_HTTP_V3_ERR_FRAME_ERROR;
}
@ -234,10 +238,6 @@ ngx_http_v3_parse_headers(ngx_connection_t *c, ngx_http_v3_parse_headers_t *st,
return rc;
}
if (st->length == 0) {
return NGX_HTTP_V3_ERR_FRAME_ERROR;
}
st->state = sw_verify;
break;