[Live-devel] [PATCH] Hex decoding for RTSP URLs

[Warren Young]

The attached patch adds unconditional URL decoding to the "urlSuffix" 
part of the URL only.

I added this so I could have spaces in file names, though it has other 
benefits.  For one thing, since RTSP specifies that header info is to be 
interpreted as UTF-8, it means we get Unicode file name support for free 
on UTF-8 based OSes.

In anticipation of any argument as to whether we should do this 
unconditionally, I refer you to RFC 2616 (HTTP 1.1) section 5.1.2, which 
says hex decoding MUST be done.  Since RTSP is specified as a kind of 
logical extension to HTTP and doesn't otherwise specify how URIs are to 
be processed, I suppose this means RTSP also must do URL hex decoding, 
even if the URL doesn't specify a file name.

I further suppose that means we should be applying URL decoding to the 
other parts of the URL.  I don't need it, but all it would take to fix 
that lack is to call urlDecode() on these other parts, too.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: live555-rtsp-url-decoding.patch
Type: text/x-patch
Size: 1159 bytes
Desc: not available
URL: <http://lists.live555.com/pipermail/live-devel/attachments/20130329/45bc8c5f/attachment.bin>