[rtmpdump] [PATCH] Better URL decoding support

Howard Chu hyc at highlandsun.com
Wed Oct 24 15:54:44 CEST 2012


NhJm wrote:
> On Tue, Oct 23, 2012 at 9:12 AM, Steven Penny <svnpenn at gmail.com
> <mailto:svnpenn at gmail.com>> wrote:
>
>     -               /* skip extension */
>     -               if (subExt && p == ext) {
>     -                       p += 4;
>     -                       pplen -= 4;
>     -                       continue;
>     -               }
>
> Please be careful not to break things.
>
> Anyways, the proper behavior is to actually *NOT* do any urldecoding. The %xx
> should be sent to the server as-is. This is what the Flash Player does and is
> the same as how normal HTTP clients handle URLs.

Correct. This patch should be discarded.



More information about the rtmpdump mailing list