diff options
| author | Maxim Dounin <mdounin@mdounin.ru> | 2020-09-29 15:54:09 +0300 |
|---|---|---|
| committer | Maxim Dounin <mdounin@mdounin.ru> | 2020-09-29 15:54:09 +0300 |
| commit | e64f7fe7c903e6994defb21db0b9667dbec7c20d (patch) | |
| tree | 1981eaf5fcdee314fbbb725951da8a489d060b82 /src/http/modules/ngx_http_proxy_module.c | |
| parent | 718d589091a1c653595f9b210bba132c43dd2c75 (diff) | |
| download | nginx-e64f7fe7c903e6994defb21db0b9667dbec7c20d.tar.gz nginx-e64f7fe7c903e6994defb21db0b9667dbec7c20d.zip | |
Proxy: error checking for array init, missed in 7716:d6a5e14aa3e4.
Found by Coverity (CID 1467637).
Diffstat (limited to 'src/http/modules/ngx_http_proxy_module.c')
| -rw-r--r-- | src/http/modules/ngx_http_proxy_module.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/http/modules/ngx_http_proxy_module.c b/src/http/modules/ngx_http_proxy_module.c index 17e4761c6..c5c9e8587 100644 --- a/src/http/modules/ngx_http_proxy_module.c +++ b/src/http/modules/ngx_http_proxy_module.c @@ -2679,7 +2679,9 @@ ngx_http_proxy_rewrite_cookie(ngx_http_request_t *r, ngx_table_elt_t *h) ngx_keyval_t *attr; ngx_http_proxy_loc_conf_t *plcf; - ngx_array_init(&attrs, r->pool, 2, sizeof(ngx_keyval_t)); + if (ngx_array_init(&attrs, r->pool, 2, sizeof(ngx_keyval_t)) != NGX_OK) { + return NGX_ERROR; + } if (ngx_http_proxy_parse_cookie(&h->value, &attrs) != NGX_OK) { return NGX_ERROR; |