<font color="#222222" face="arial, sans-serif"><span style="font-size:14px"><div>Thanks~ And in the original patch, I introduced a variable c to save the connection. I did</div><div>this because I think it may be not safe to continue using the variable r after calling </div>
<div>ngx_http_upstream_finalize_request, in which the request struct and pool may be freed.</div></span></font><br><div class="gmail_quote">2013/3/14 Maxim Dounin <span dir="ltr"><<a href="mailto:mdounin@mdounin.ru" target="_blank">mdounin@mdounin.ru</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello!<br>
<div><br>
On Thu, Mar 14, 2013 at 06:44:48AM +0800, lanshun zhou wrote:<br>
<br>
> Because there is chance to call ngx_http_upstream_finalize_request in<br>
> function ngx_http_upstream_connect or it's sub calls, and i think this<br>
> makes no difference with failed resolution~<br>
<br>
</div>Yes, you are right.  I'v committed slightly modified version of<br>
the patch.<br>
<div><div><br>
--<br>
Maxim Dounin<br>
<a href="http://nginx.org/en/donation.html" target="_blank">http://nginx.org/en/donation.html</a><br>
<br>
_______________________________________________<br>
nginx-devel mailing list<br>
<a href="mailto:nginx-devel@nginx.org" target="_blank">nginx-devel@nginx.org</a><br>
<a href="http://mailman.nginx.org/mailman/listinfo/nginx-devel" target="_blank">http://mailman.nginx.org/mailman/listinfo/nginx-devel</a><br>
</div></div></blockquote></div><br>