<div dir="ltr">Hi Maxim,<div><br></div><div>I'm sorry, it's my mistake I missed that</div><div><br></div><div>sin = &sockaddr[d].sockaddr_in;<br></div><div><br></div><div>and</div><div><br></div><div>dst[d].sockaddr = (struct sockaddr *) sin;<br></div><div><br></div><div>Sorry for the confusion.</div><div><br></div><div>Best,</div><div>Lubos</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Mar 6, 2022 at 2:59 AM Maxim Dounin <<a href="mailto:mdounin@mdounin.ru">mdounin@mdounin.ru</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello!<br>
<br>
On Thu, Mar 03, 2022 at 02:11:26AM +0100, Lubos Uhliarik wrote:<br>
<br>
>  # HG changeset patch<br>
> # User Lubos Uhliarik <<a href="mailto:luhliari@redhat.com" target="_blank">luhliari@redhat.com</a>><br>
> # Date 1646269812 -3600<br>
> #      Thu Mar 03 02:10:12 2022 +0100<br>
> # Node ID 317e1e4b0c7343c49e0e13fc59ac75a565521b67<br>
> # Parent  a736a7a613ea6e182ff86fbadcb98bb0f8891c0b<br>
> Fix resource leak - sockaddr is not properly freed<br>
> <br>
> sockaddr variable is allocated by ngx_resolver_calloc function but then it<br>
> is<br>
> going out of scope leaking the storage it points to.<br>
> <br>
> diff -r a736a7a613ea -r 317e1e4b0c73 src/core/ngx_resolver.c<br>
> --- a/src/core/ngx_resolver.c   Tue Feb 08 17:35:27 2022 +0300<br>
> +++ b/src/core/ngx_resolver.c   Thu Mar 03 02:10:12 2022 +0100<br>
> @@ -4260,6 +4260,8 @@<br>
>      }<br>
>  #endif<br>
> <br>
> +    ngx_resolver_free(r, sockaddr);<br>
> +<br>
>      return dst;<br>
>  }<br>
<br>
Could you please clarify why do you think there is a leak?  Note <br>
that sockaddr is referenced in the dst array, which is being <br>
returned.<br>
<br>
-- <br>
Maxim Dounin<br>
<a href="http://mdounin.ru/" rel="noreferrer" target="_blank">http://mdounin.ru/</a><br>
_______________________________________________<br>
nginx-devel mailing list -- <a href="mailto:nginx-devel@nginx.org" target="_blank">nginx-devel@nginx.org</a><br>
To unsubscribe send an email to <a href="mailto:nginx-devel-leave@nginx.org" target="_blank">nginx-devel-leave@nginx.org</a><br>
<br>
</blockquote></div>