[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Bug-wget] [PATCH] Regression since wget 1.10: no_prefix function is *ba

From: Tim Rühsen
Subject: [Bug-wget] [PATCH] Regression since wget 1.10: no_prefix function is *bad*
Date: Sat, 11 May 2013 17:22:18 +0200
User-agent: KMail/1.13.7 (Linux/3.8-1-amd64; KDE/4.8.4; x86_64; ; )

Hi Martin,

having an abort() without a message is simply a big waste of time for any 
developer who stumbles upon it.

Since the init code of Wget has to be rewritten anyways, i provide the fastest 
solution right now: increasing the buffer size and printing a message before 
Wget aborts.

And yes, the whole issue is hell stupid...

Regards, Tim

Am Freitag, 10. Mai 2013 schrieb Marlin Frickenschmidt:
> Hello dear wget maintainers,
> I want to report a bug of sorts. It is not a direct bug that impedes the
> operation of wget to normal users, but one which basically makes it
> impossible to add more command-line options to wget without sooner or
> later making wget suddenly SIGABRT - without any inclination to show a
> proper error whatsover.
> The reason for this is the "no_prefix" function, which is supposed to
> prepend the string "no-" to a given string (for disabling certain
> command line options). The function makes the assumption that the total
> length of all command line option names together won't exceed 1024
> characters, because the buffer storing the strings for all the
> "no-"-prefixed command line options is only chosen that large. And so if
> it gets too big, there is just a silent abort(), no error message, no
> nothing.
> Why don't we use standard functions like asprintf to prepend strings,
> and instead build an own, completely broken function for it? Perhaps
> there are good reasons for building the string yourself instead of using
> the standard library; I don't know. In any case, there is no excuse for
> silently abort()ing without an error message. That is the only thing I
> am grumpy about, really...
> I hope this can be fixed in the next release for the sake of all wget
> developers around. I just spent two hours debugging this, and really
> couldn't believe my eyes when I found the reason for it. It is hell stupid.
> Cheers,
> Marlin

Attachment: 0003-warn-before-abort-in-main.c-no_prefix.patch
Description: Text Data

reply via email to

[Prev in Thread] Current Thread [Next in Thread]