[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:13:51 UTC 2007


A NOTE has been added to this issue. 
====================================================================== 
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:                     assigned
====================================================================== 
Date Submitted:             09-24-2007 05:43 PDT
Last Modified:              09-24-2007 06:13 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 

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                          
======================================================================




More information about the busybox-cvs mailing list