Matthew Dempsky
September 16, 2009 12:30PM
On Wed, Sep 16, 2009 at 3:35 AM, Maxim Dounin <mdounin@mdounin.ru> wrote:
> Patch looks correct for me.  It looks a bit fragile though,
> probably we need a bit more bulletproof code here.

Would you mind elaborating on what you think is fragile about it?
E.g., how would you rather this bug be patched?

> More generally - resolver known to leak, and probably requires
> code audit.  It would be fine if you look into it.  I believe
> Artem Bokhan will help with testing (cc'd as I'm not sure he is on
> English list).

Great. I'll spend some time looking at the rest of the code then.

Thanks.
Subject Author Posted

ngx_resolver.c leaks memory?

Matthew Dempsky September 16, 2009 01:36AM

Re: ngx_resolver.c leaks memory?

Maxim Dounin September 16, 2009 06:42AM

Re: ngx_resolver.c leaks memory?

Matthew Dempsky September 16, 2009 12:30PM

Re: ngx_resolver.c leaks memory?

Maxim Dounin September 17, 2009 06:44AM

Re: ngx_resolver.c leaks memory?

Igor Sysoev September 17, 2009 07:52AM

Re: ngx_resolver.c leaks memory?

Maxim Dounin September 17, 2009 09:34AM

Re: ngx_resolver.c leaks memory?

Igor Sysoev September 16, 2009 10:04AM



Sorry, only registered users may post in this forum.

Click here to login

Online Users

Guests: 112
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 500 on July 15, 2024
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready