[FFmpeg-devel] [PATCH] Properly handle cookies that specify sub-domain where the URL.

Carl Eugen Hoyos cehoyos at ag.or.at
Wed Jan 22 14:23:10 CET 2014


Eli Kara <eli <at> algotec.co.il> writes:

> +                // the domain in the URL should be at least 
> the size of our cookie domain. correctly support sub-domains AND
> the master domain
> +                // by not comparing the dot at all (ex: 
> domain=bar.com and cdomain=.bar.com)

Please try to fix the length of the comment lines.

> +				int leading_dot = (param[7] == '.') ? 1 : 0;

tabs cannot be committed to the FFmpeg repository, 
please remove them and consider using tools/patcheck.

I suspect you can remove the additional variable (or 
the "?").

Carl Eugen



More information about the ffmpeg-devel mailing list