On Mon, Nov 20, 2017 at 01:31:06PM -0800, Dale Curtis wrote:
> On Mon, Nov 20, 2017 at 12:34 PM, Michael Niedermayer <
> mich...@niedermayer.cc> wrote:
>
> > On Mon, Nov 20, 2017 at 12:01:11PM -0800, Dale Curtis wrote:
> > > Derp, that's the wrong comment for this; I meant that for another thread.
On Mon, Nov 20, 2017 at 12:34 PM, Michael Niedermayer <
mich...@niedermayer.cc> wrote:
> On Mon, Nov 20, 2017 at 12:01:11PM -0800, Dale Curtis wrote:
> > Derp, that's the wrong comment for this; I meant that for another thread.
> > Returning an error for optional user data seems a bit harsh, but i
On Mon, Nov 20, 2017 at 12:01:11PM -0800, Dale Curtis wrote:
> Derp, that's the wrong comment for this; I meant that for another thread.
> Returning an error for optional user data seems a bit harsh, but if that's
> what you want, I'm happy to change it. Please let me know if this is the
> route yo
Derp, that's the wrong comment for this; I meant that for another thread.
Returning an error for optional user data seems a bit harsh, but if that's
what you want, I'm happy to change it. Please let me know if this is the
route you want to take.
- dale
On Mon, Nov 20, 2017 at 11:55 AM, Dale Curti
On Fri, Nov 17, 2017 at 6:22 PM, Michael Niedermayer wrote:
>
> Not sure what is best but
> throwing part of the version silently away is not correct
> most likely erroring out and asking for a sample video to be uploaded
> would make sense if such a file is encountered
I'm using the same workar
On Fri, Nov 17, 2017 at 04:07:42PM -0800, Dale Curtis wrote:
> decode_user_data() attempts to create an integer |build|
> value with 8 bits of spacing for 3 components. However
> each component is an int32_t, so shifting each component
> is undefined for values outside of the 8 bit range.
>
> This
decode_user_data() attempts to create an integer |build|
value with 8 bits of spacing for 3 components. However
each component is an int32_t, so shifting each component
is undefined for values outside of the 8 bit range.
This patch simply clamps input to 8-bits per component.
Signed-off-by: Dale