<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">value of csp is used in YUVInput(info) / Y4MInput(info) before x265_check_params() getting called.</span><br><div class="gmail_extra"><br></div><div class="gmail_extra">
<br><div class="gmail_quote">On Tue, Mar 4, 2014 at 1:57 AM, Steve Borho <span dir="ltr"><<a href="mailto:steve@borho.org" target="_blank">steve@borho.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">On Mon, Mar 3, 2014 at 5:19 AM,  <<a href="mailto:sagar@multicorewareinc.com">sagar@multicorewareinc.com</a>> wrote:<br>
> # HG changeset patch<br>
> # User Sagar Kotecha <<a href="mailto:sagar@multicorewareinc.com">sagar@multicorewareinc.com</a>><br>
> # Date 1393845552 -19800<br>
> #      Mon Mar 03 16:49:12 2014 +0530<br>
> # Node ID 631d6db7f96980aa032301a2cafa21177cc524a2<br>
> # Parent  288a83d7e28999798859eba6b2f38c952cac7547<br>
> param: Add validation check for invalid colorspace to prevent crash<br>
><br>
> diff -r 288a83d7e289 -r 631d6db7f969 source/common/param.cpp<br>
> --- a/source/common/param.cpp   Sun Mar 02 18:57:46 2014 -0600<br>
> +++ b/source/common/param.cpp   Mon Mar 03 16:49:12 2014 +0530<br>
> @@ -557,7 +557,12 @@<br>
>          p->rc.qp = atoi(value);<br>
>          p->rc.rateControlMode = X265_RC_CQP;<br>
>      }<br>
> -    OPT("input-csp") p->internalCsp = parseName(value, x265_source_csp_names, bError);<br>
> +    OPT("input-csp")<br>
> +    {<br>
> +       p->internalCsp = parseName(value, x265_source_csp_names, bError);<br>
<br>
> +       if(p->internalCsp < X265_CSP_I420 || p->internalCsp > X265_CSP_I444)<br>
<br>
</div>white-space<br>
<br>
> +           bError |= true;<br>
<br>
why do this range check here? x265_check_params() is already<br>
validating that the color space is one of the two that are currently<br>
supported.<br>
<div class=""><br>
> +    }<br>
>      OPT("me")        p->searchMethod = parseName(value, x265_motion_est_names, bError);<br>
>      OPT("cutree")    p->rc.cuTree = atobool(value);<br>
>      OPT("vui")<br>
</div>> _______________________________________________<br>
> x265-devel mailing list<br>
> <a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
> <a href="https://mailman.videolan.org/listinfo/x265-devel" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
--<br>
Steve Borho<br>
_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</font></span></blockquote></div><br></div></div>