curl-library
Re: [PATCH 1/8] tests: fix memleak in server/resolve.c
From: Daniel Stenberg <daniel_at_haxx.se>
Date: Mon, 10 Nov 2014 14:23:16 +0100 (CET)
Date: Mon, 10 Nov 2014 14:23:16 +0100 (CET)
On Mon, 10 Nov 2014, Peter Wu wrote:
>> ... this needs to be (freeaddrinfo)(ai) or it goes bananas when built with
>> memdebug emabled. I fixed it in my local branch so no need to repost
>> anything.
>
> Ah, I missed the comment above that code. This is quite unexpected behavior,
> but that is what you get for overriding standard functions I guess.
Well, I got a linker error immediately so it was quite obvious to me... =) And
I would argue that it (== defining functions) is a standard way to do basic
memory debugging that works cross-platform without additional tools.
-- / daniel.haxx.se ------------------------------------------------------------------- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.htmlReceived on 2014-11-10