Re: [libmicrohttpd] memleak in digestauth.c @ check_argument_match()

[Christian Grothoff]

Thanks, fixed as suggested in SVN 35864 (and yes, very odd that this
wasn't caught by static analysis or other runs with valgrind before).

-Christian

On 06/04/2015 01:22 PM, Andreas Wehrmann wrote:
> Hi!
> 
> I was checking a test app in valgrind and much to my surprise it was 
> complaining about a memleak in libmicrohttpd.
> In check_argument_match() a buffer is allocated using strdup() but freed 
> nowhere.
> 
> I wouldn't have noticed this leak if it wasn't for valgrind because if a URI 
> is requested without any arguments (my usual case)
> the buffer that gets allocated has 'only' a length of 1 byte, thus memory 
> usage will build up very slowly over time.
> 
> The patch attached fixes it.
> Btw. the indentation of that file is a mess, tabs and spaces are mixed :-|
> 
> Regards,
> Andreas
>

0xE29FC3CC.asc
Description: application/pgp-keys

signature.asc
Description: OpenPGP digital signature