<div dir="ltr"><div><div><div><div><div><div>Hello, Nginx developers,<br><br></div>I been working recently on NGinx, as part of my job .<br><br></div>And I see some small issues as this one :<br></div><br></div>This call to *ngx_close_connection* is useless as it was already made
18 lines before in the code .<br></div><div>Same code is present in 1.2<br></div><div><br></div>Regards,<br></div>Filipe<br><br><div><div><div><div>----<br></div><div></div><div><br>diff -r 8222ca034980 src/mail/ngx_mail_auth_http_module.c<br>
<div><div><div>--- a/src/mail/ngx_mail_auth_http_module.c    Tue Apr 02 12:34:39 2013 +0000<br>
+++ b/src/mail/ngx_mail_auth_http_module.c    Thu May 02 15:46:42 2013 +0200<br>@@ -696,7 +696,6 @@<br> <br>                     p = ngx_pnalloc(s->connection->pool, ctx->err.len);<br>                     if (p == NULL) {<br>

-                        ngx_close_connection(ctx->peer.connection);<br>                         ngx_destroy_pool(ctx->pool);<br>                         ngx_mail_session_internal_server_error(s);<br>                         return;<br>

<br></div></div></div></div></div></div></div></div>