<div dir="ltr"><div><div><div><div><div>Hi,<br></div>Please find attach this patch as a mercurial changeset, <br></div>It's based on the current trunk version.<br></div></div><div><br></div>Regards,<br></div>
Filipe</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/5/8 Valentin V. Bartenev <span dir="ltr"><<a href="mailto:vbart@nginx.com" target="_blank">vbart@nginx.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Thursday 02 May 2013 18:01:29 Filipe Da Silva wrote:<br>
> Hello, Nginx developers,<br>
><br>
> I been working recently on NGinx, as part of my job .<br>
><br>
> And I see some small issues as this one :<br>
><br>
> This call to *ngx_close_connection* is useless as it was already made 18<br>
> lines before in the code .<br>
> Same code is present in 1.2<br>
><br>
> Regards,<br>
> Filipe<br>
><br>
> ----<br>
><br>
> diff -r 8222ca034980 src/mail/ngx_mail_auth_http_module.c<br>
> --- a/src/mail/ngx_mail_auth_http_module.c    Tue Apr 02 12:34:39 2013<br>
> +0000 +++ b/src/mail/ngx_mail_auth_http_module.c    Thu May 02 15:46:42<br>
> 2013 +0200 @@ -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>The patch looks good for me. It would be nice if you provide it as<br>
a mercurial changeset for pushing.<br>
<br>
  wbr, Valentin V. Bartenev<br>
</blockquote></div><br></div>