[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
>
>
0003-warn-before-abort-in-main.c-no_prefix.patch
Description: Text Data