[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