View Issue Details

IDProjectCategoryView StatusLast Update
0002776CentOS-5wgetpublic2009-10-17 22:45
Reporterinejge Assigned To 
Status resolvedResolutionfixed 
Product Version5.1 
Fixed in Version5.4 
Summary0002776: wget aborts if invoked with --no-clobber when it encounters existing file
DescriptionIf wget (1.10.2-7.el5) is run with --no-clobber, it will abort after determining that a file already exists locally. No further error is displayed. This can always be reproduced:

    for i in 1 2;do wget -nc;done

If the first retrieval was successful, the second will fail.
Additional InformationAccording to my quick look at wget sources as shipped with CentOS, this occurs because of code rearrangement in the massive patch (wget-1.10.2-to11.patch) which is applied to the original. In the patched version, the test for "noclobber" is no longer within http_loop(), but one level deeper, in gethttp(). When this function returns RETROK after determining local existence of a file, a switch statement doesn't know how to deal with that value, and aborts.

I created a quick-and-dirty patch which changes the return value from RETROK to RETRUNNEEDED and should be applied after wget-1.10.2-to11.patch. After applying that patch, wget no longer aborts in the circumstances described above. Someone who knows more about wget should check for memory leaks and similar.
Tagsfixed in 5.4


2008-04-04 10:35


wget-1.10.2-noclobber.patch (347 bytes)   
--- wget-1.10.2/src/http.c.old	2008-04-04 11:51:40.000000000 +0200
+++ wget-1.10.2/src/http.c	2008-04-04 11:54:28.000000000 +0200
@@ -1760,7 +1760,7 @@
           if (has_html_suffix_p (hs->local_file))
             *dt |= TEXTHTML;
-          return RETROK;
+          return RETRUNNEEDED;
       else if (!ALLOW_CLOBBER)
wget-1.10.2-noclobber.patch (347 bytes)   


2008-04-04 13:19


You should report that upstream, if you think this should be changed, as the patch comes from there (we didn't change the sources when rebuilding).


2009-02-18 16:45


Still interested in getting that bug fixed?


2009-07-03 14:00

reporter   ~0009563

Yes, please, this is a very annoying bug to me.

I also saw it here,

However it seems to work well in debian...


2009-07-03 15:02


Will be fixed in 5.4

Issue History

Date Modified Username Field Change
2008-04-04 10:35 inejge New Issue
2008-04-04 10:35 inejge File Added: wget-1.10.2-noclobber.patch
2008-04-04 13:19 user430 Note Added: 0007093
2009-02-18 16:45 user430 Note Added: 0008742
2009-07-03 14:00 andreu Note Added: 0009563
2009-07-03 15:02 user430 Note Added: 0009565
2009-07-03 15:03 user430 Status new => confirmed
2009-07-03 15:03 user430 Tag Attached: fixed in 5.4
2009-10-17 22:45 user430 Status confirmed => resolved
2009-10-17 22:45 user430 Fixed in Version => 5.4
2009-10-17 22:45 user430 Resolution open => fixed