[PATCH] winealsa: snd_card_get_name uses strdup (Valgrind).
Michael Stefaniuc
mstefani at redhat.com
Wed Feb 1 11:27:03 CST 2012
Lauri Kenttä wrote:
> On 2012-02-01 17:00, Christian Costa wrote:
>> In that case you don't need the if (!err) condition to free the buffer
>> otherwise you leak the string return by strdup.
>
> Yeah, I didn't think this through and screwed up. :P
>
> I also noticed now that Jörg Höhle already deemed this strdup solution
> ugly in his previous try. Luckily Michael Stefaniuc sent a finer patch
> #83376.
Still not good enough. But at least we forced Alexandre to fix it himself ;)
bye
michael
More information about the wine-devel
mailing list