Subject: Re: [PATCH] ares_parse_soa_reply: Do not leak rr_name on allocation failure

Re: [PATCH] ares_parse_soa_reply: Do not leak rr_name on allocation failure

From: David Drysdale <drysdale_at_google.com>
Date: Tue, 20 May 2014 16:56:33 +0100

The updated code change looks OK to me (but I've not actually exercised the
new arm).

D.

On Tue, May 20, 2014 at 3:18 PM, Jakub Hrozek <jhrozek_at_redhat.com> wrote:

> On Tue, May 20, 2014 at 02:43:42PM +0100, David Drysdale wrote:
> > Don't you also need to set status = ARES_ENOMEM; before the goto
> > failed_stat ?
>
> If course I do, thanks!
>
> New patch is attached.
>
Received on 2014-05-20