[BusyBox 0001508]: wget don't support DLNA enabled HTTP servers (and their special headers)

bugs at busybox.net bugs at busybox.net
Mon Sep 24 13:52:39 UTC 2007


The following issue has been CLOSED 
====================================================================== 
http://busybox.net/bugs/view.php?id=1508 
====================================================================== 
Reported By:                sloeuillet
Assigned To:                BusyBox
====================================================================== 
Project:                    BusyBox
Issue ID:                   1508
Category:                   Standards Compliance
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Resolution:                 open
Fixed in Version:           
====================================================================== 
Date Submitted:             09-24-2007 05:43 PDT
Last Modified:              09-24-2007 06:52 PDT
====================================================================== 
Summary:                    wget don't support DLNA enabled HTTP servers (and
their special headers)
Description: 
In the middle of classic HTTP headers that any web server would send, a
DLNA server would send this special header :
transfermode.dlna.org: Interactive

Yes, you see well, those are dots.

Looking at wget.c source code, more specifically gethdr() method, we could
see :
http://busybox.net/cgi-bin/viewcvs.cgi/trunk/busybox/networking/wget.c?rev=19557&view=markup

	/* convert the header name to lower case */
	for (s = buf; isalnum(*s) || *s == '-'; ++s)
		*s = tolower(*s);

	/* verify we are at the end of the header name */
	if (*s != ':')
		bb_error_msg_and_die("bad header line: %s", buf);

As you see, it would go to the second test once the first '.' is
encountered, and would display the message.

Of course, it would fail to download the file


It works ok with the standard wget utility.
As DLNA is a standard protocol on top of HTTP, we could suppose this is
standard.

Could you please fix it to allow dots too ?
====================================================================== 

---------------------------------------------------------------------- 
 sloeuillet - 09-24-07 06:13  
---------------------------------------------------------------------- 
Patch against 1.01 but looking at the code, it didn't change much since
then and should apply smoothly 

---------------------------------------------------------------------- 
 vda - 09-24-07 06:52  
---------------------------------------------------------------------- 
Fixed in svn. Thanks. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
09-24-07 05:43  sloeuillet     New Issue                                    
09-24-07 05:43  sloeuillet     Status                   new => assigned     
09-24-07 05:43  sloeuillet     Assigned To               => BusyBox         
09-24-07 06:13  sloeuillet     Note Added: 0002787                          
09-24-07 06:52  vda            Status                   assigned => closed  
09-24-07 06:52  vda            Note Added: 0002788                          
======================================================================




More information about the busybox-cvs mailing list