Re: [Bug-wget] Memory leak in idn_encode; Valgrind suppression file

[Hubert Tarasiuk]

W dniu 07.04.2015 o 00:11, Ángel González pisze:
> On 06/04/15 22:15, Hubert Tarasiuk wrote:
>> We should probably also fix this comment:
>>>         /* sXXXav : free new when needed ! */
>> As it presumably mentions the problem that we are going to repair.
> I thought it refered to idna_to_ascii_8z sometimes allocating memory on
> error (that's
> why I didn't touch it), but it may as well refer to this.
> 
That would make more sense based on the location of the comment.
I looked at the current source code for idna_to_ascii_8z (
http://www.gnu.org/software/libidn/doxygen/idna_8c_source.html#l00572 )
and if I am not mistaken, it should not happen that the *output will be
set to allocated memory, and the function will fail.
(I did not find anything concerning this problem in the manual of this
function.)
Maybe the comment is out of date? (Commited in July 2008.) Is the author
of it on this list?

signature.asc
Description: OpenPGP digital signature