<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial">Is this ok? Or return a 500?<br><br><div># HG changeset patch</div><div># User FengGu <flygoast@126.com></div><div># Date 1393211386 -28800</div><div># Node ID 8f1937a9f3f632cf060d18053f153bbf6097f5ba</div><div># Parent  f47c844acbd453c05174200af8df132c33171b35</div><div>Fixed segmentation fault with error_page for 400 to named location.</div><div><br></div><div>When using error_page for code 400 to named location, the bad request's</div><div>uri would be { 0, NULL }, it would result in segmentation fault in</div><div>ngx_http_index_handler() and so on.</div><div><br></div><div>diff -r f47c844acbd4 -r 8f1937a9f3f6 src/http/ngx_http_core_module.c</div><div>--- a/src/http/ngx_http_core_module.c   Sat Feb 22 12:08:31 2014 +0400</div><div>+++ b/src/http/ngx_http_core_module.c   Mon Feb 24 11:09:46 2014 +0800</div><div>@@ -2647,6 +2647,11 @@</div><div>                 continue;</div><div>             }</div><div><br></div><div>+            if (r->uri.len == 0) {</div><div>+                ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST);</div><div>+                return NGX_DONE;</div><div>+            }</div><div>+</div><div>             ngx_log_debug3(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,</div><div>                            "using location: %V \"%V?%V\"",</div><div>                            name, &r->uri, &r->args);</div><br><div></div><div id="divNeteaseMailCard"></div><br><pre><br>At 2014-02-22 01:37:54,"Maxim Dounin" <mdounin@mdounin.ru> wrote:
>Hello!
>
>On Sat, Feb 22, 2014 at 12:40:35AM +0800, flygoast wrote:
>
>> # HG changeset patch
>> 
>> # User FengGu <flygoast@126.com>
>> # Date 1392981015 -28800
>> #          Fri Feb 21 19:10:15 2014 +0800
>> # Node ID 2dd8fd75f1646336b21cb2f4506f1d45b8771e56
>> # Parent  545a4d393e2fb8d5448dad89d25a110fa72e71d1
>> Fixed segmentation fault with error_page for 400 to named location
>> 
>> 
>> When using error_page for code 400 to named location, the bad request's uri
>> would be { 0, NULL }, it would result in segmentation fault in
>> ngx_http_index_handler() and so on.
>> 
>> 
>> diff -r 545a4d393e2f -r 2dd8fd75f164 src/http/ngx_http_core_module.c
>> --- a/src/http/ngx_http_core_module.c   Thu Feb 20 17:27:09 2014 +0400
>> +++ b/src/http/ngx_http_core_module.c   Fri Feb 21 19:10:15 2014 +0800
>> @@ -2651,6 +2651,12 @@
>>                             "using location: %V \"%V?%V\"",
>>                             name, &r->uri, &r->args);
>> 
>> 
>> +            if (r->uri.len == 0) {
>> +                ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
>> +                              "using \"/\" as uri for bad request");
>> +                ngx_str_set(&r->uri, "/");
>> +            }
>> +
>>              r->internal = 1;
>>              r->content_handler = NULL;
>>              r->uri_changed = 0;
>
>This doesn't looks like a good solution.
>I would rather return an error in such a case.
>
>-- 
>Maxim Dounin
>http://nginx.org/
>
>_______________________________________________
>nginx-devel mailing list
>nginx-devel@nginx.org
>http://mailman.nginx.org/mailman/listinfo/nginx-devel
</pre></div><br><br><span title="neteasefooter"><span id="netease_mail_footer"></span></span>