Re: [Bug-wget] [PATCH] the libidn problem

[Tim Ruehsen]

On Wednesday 01 July 2015 13:44:08 Ander Juaristi wrote:
> On 07/01/2015 01:28 PM, Tim Ruehsen wrote:
> > Definitely not. If quote() doesn't work, Wget wouldn't work at all.
> 
> I wasn't implying quote() is broken. As you say, if it was broken, Wget
> would not work at all, and it works alright in my box. Also, 'make check'
> does well. It only segfaults when I input 0xFC, that's what I meant from
> the beginning. And since _utf8_is_valid() should work with even invalid
> sequences...

Sorry Ander. My mistake :-(
Inputting "\xFC" to idn_encode() crashes quote() at where you say.

Sorry for the confusion.

> 
> 'gdb' traces it to quotearg.c:343:
> 
>      for (i = 0;  ! (argsize == SIZE_MAX ? arg[i] == '\0' : i == argsize); 
> i++)
> 
> I have no gcc optimizations applied, so it should be pretty accurate. Will
> look further, don't have time now.

Maybe I can look at it later this afternoon...

Tim