bugGNU Wget - Bugs: bug #40426, wget hangs with -r and -O -

 
 

bug #40426: wget hangs with -r and -O -

Submitted by:  None
Submitted on:  Wed 30 Oct 2013 11:51:48 AM UTC  
 
Category:  Crash/Freeze/Infloop Severity:  3 - Normal
Priority:  5 - Normal Status:  Fixed
Privacy:  Public Assigned to:  None
Originator Name:  Antonio Originator Email:  -email is unavailable-
Open/Closed:  Closed Release:  1.13.4
Operating System:  None Reproducibility:  Every Time
Fixed Release:  None Planned Release:  None
Regression:  None Work Required:  None
Patch Included:  No

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Wed 09 Sep 2015 07:36:51 AM UTC, comment #3:

Well, it has been 'fixed' by not allowing -r in combination with -O- in wget1.x.

Wget2 already works with -r -O-.

Tim Ruehsen <rockdaboot>
Project Administrator
Tue 08 Sep 2015 07:20:32 AM UTC, comment #2:

This was eventually merged on 2015-08-15, commit 12bae50b28fa20f9ffa3d5b0e88f4ebf51fa6864.

Please, check whether the issue is effectively fixed, and close this.

Ander Juaristi <ajuaristi>
Sat 14 Mar 2015 05:19:43 PM UTC, comment #1:

I thought this is still a valid bug and wrote my patch. I'm surprised to find Calore's patch in attached files and that patch is finished in Nov 2013!
Anyway, my patch is easier to am.
https://savannah.gnu.or

Lucas Yu <lucasyu>
Wed 30 Oct 2013 11:51:48 AM UTC, original submission:

# wget -r -O - http://www.blankwebsite.com

hangs forever, I can verify it with a strace. If I do this:

while read R
do
wget -r -O - http://www.blankwebsite.com
done < smallfile
with smallfile containing a single line, the command exits in a few seconds.

I tried also with

wget -r -O - localhost/test.html
with an empty test.html file, same results.

Everything runs fine changing -O - with -O myfile or removing -r.

Asked on stackoverflow:
http://stackoverflow.com/questions/19681316/wget-hangs-with-r-and-o

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #33344:  Fix-hang-with-r-O-.patch added by lucasyu (764B - application/octet-stream - Fix this by simply forbidding -r -O- combination)
file #29593:  wget-gitpatch-recursivecheck.diff added by None (948B - application/octet-stream - Patch attached that will check the "-r -O FILE" combination and only allow it if FILE is a regular file.)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rockdaboot (Posted a comment)
  • -email is unavailable- added by ajuaristi (Posted a comment)
  • -email is unavailable- added by lucasyu (Updated the item)
  •  

     

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-09-16 rockdaboot StatusNone => Fixed
        Open/ClosedOpen => Closed
    2015-03-14 lucasyu Attached File- => Added Fix-hang-with-r-O-.patch, #33344
    2013-11-11 None Attached File- => Added wget-gitpatch-recursivecheck.diff, #29593

    Back to the top


    Powered by Savane 3.3