Mercurial > hg > nginx

comparison src/http/ngx_http_request.c @ 7753:2fec22332ff4

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Removed extra allocation for r->uri. The ngx_http_parse_complex_uri() function cannot make URI longer and does not null-terminate URI, so there is no need to allocate an extra byte. This allocation appears to be a leftover from changes in 461:a88a3e4e158f (0.1.5), where null-termination of r->uri and many other strings was removed.
author Maxim Dounin <mdounin@mdounin.ru>
date Thu, 10 Dec 2020 20:09:39 +0300
parents 8989fbd2f89a
children 4e141d0816d4 f61d347158d0
comparison
equal deleted inserted replaced
7752:8989fbd2f89a 7753:2fec22332ff4
1228 1228
1229 if (r->empty_path_in_uri) { 1229 if (r->empty_path_in_uri) {
1230 r->uri.len++; 1230 r->uri.len++;
1231 } 1231 }
1232 1232
1233 r->uri.data = ngx_pnalloc(r->pool, r->uri.len + 1); 1233 r->uri.data = ngx_pnalloc(r->pool, r->uri.len);
1234 if (r->uri.data == NULL) { 1234 if (r->uri.data == NULL) {
1235 ngx_http_close_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR); 1235 ngx_http_close_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR);
1236 return NGX_ERROR; 1236 return NGX_ERROR;
1237 } 1237 }
1238 1238