diff options
| author | Roman Arutyunyan <arut@nginx.com> | 2020-07-03 12:05:05 +0300 |
|---|---|---|
| committer | Roman Arutyunyan <arut@nginx.com> | 2020-07-03 12:05:05 +0300 |
| commit | 8d1875d39ab60a6494a1d3b4be34fc35213d74df (patch) | |
| tree | 207b9abd01ce7c00c1c8a27d82634e5cdff030be /src | |
| parent | 3a9874c4ab00d443c66e175502466b0105e8f3be (diff) | |
| download | nginx-8d1875d39ab60a6494a1d3b4be34fc35213d74df.tar.gz nginx-8d1875d39ab60a6494a1d3b4be34fc35213d74df.zip | |
HTTP/3: simplifed handling ngx_http_v3_parse_literal() return code.
Diffstat (limited to 'src')
| -rw-r--r-- | src/http/v3/ngx_http_v3_parse.c | 77 |
1 files changed, 21 insertions, 56 deletions
diff --git a/src/http/v3/ngx_http_v3_parse.c b/src/http/v3/ngx_http_v3_parse.c index 8b1f745fe..4e0532781 100644 --- a/src/http/v3/ngx_http_v3_parse.c +++ b/src/http/v3/ngx_http_v3_parse.c @@ -624,17 +624,12 @@ ngx_http_v3_parse_header_lri(ngx_connection_t *c, case sw_value: rc = ngx_http_v3_parse_literal(c, &st->literal, ch); - - if (rc == NGX_DONE) { - st->value = st->literal.value; - goto done; - } - - if (rc != NGX_AGAIN) { + if (rc != NGX_DONE) { return rc; } - break; + st->value = st->literal.value; + goto done; } return NGX_AGAIN; @@ -703,17 +698,12 @@ ngx_http_v3_parse_header_l(ngx_connection_t *c, case sw_name: rc = ngx_http_v3_parse_literal(c, &st->literal, ch); - - if (rc == NGX_DONE) { - st->name = st->literal.value; - st->state = sw_value_len; - break; - } - - if (rc != NGX_AGAIN) { + if (rc != NGX_DONE) { return rc; } + st->name = st->literal.value; + st->state = sw_value_len; break; case sw_value_len: @@ -741,17 +731,12 @@ ngx_http_v3_parse_header_l(ngx_connection_t *c, case sw_value: rc = ngx_http_v3_parse_literal(c, &st->literal, ch); - - if (rc == NGX_DONE) { - st->value = st->literal.value; - goto done; - } - - if (rc != NGX_AGAIN) { + if (rc != NGX_DONE) { return rc; } - break; + st->value = st->literal.value; + goto done; } return NGX_AGAIN; @@ -876,17 +861,12 @@ ngx_http_v3_parse_header_lpbi(ngx_connection_t *c, case sw_value: rc = ngx_http_v3_parse_literal(c, &st->literal, ch); - - if (rc == NGX_DONE) { - st->value = st->literal.value; - goto done; - } - - if (rc != NGX_AGAIN) { + if (rc != NGX_DONE) { return rc; } - break; + st->value = st->literal.value; + goto done; } return NGX_AGAIN; @@ -1306,17 +1286,12 @@ ngx_http_v3_parse_header_inr(ngx_connection_t *c, case sw_value: rc = ngx_http_v3_parse_literal(c, &st->literal, ch); - - if (rc == NGX_DONE) { - st->value = st->literal.value; - goto done; - } - - if (rc != NGX_AGAIN) { + if (rc != NGX_DONE) { return rc; } - break; + st->value = st->literal.value; + goto done; } return NGX_AGAIN; @@ -1382,17 +1357,12 @@ ngx_http_v3_parse_header_iwnr(ngx_connection_t *c, case sw_name: rc = ngx_http_v3_parse_literal(c, &st->literal, ch); - - if (rc == NGX_DONE) { - st->name = st->literal.value; - st->state = sw_value_len; - break; - } - - if (rc != NGX_AGAIN) { + if (rc != NGX_DONE) { return rc; } + st->name = st->literal.value; + st->state = sw_value_len; break; case sw_value_len: @@ -1420,17 +1390,12 @@ ngx_http_v3_parse_header_iwnr(ngx_connection_t *c, case sw_value: rc = ngx_http_v3_parse_literal(c, &st->literal, ch); - - if (rc == NGX_DONE) { - st->value = st->literal.value; - goto done; - } - - if (rc != NGX_AGAIN) { + if (rc != NGX_DONE) { return rc; } - break; + st->value = st->literal.value; + goto done; } return NGX_AGAIN; |