Re: [FFmpeg-devel] [PATCH, v3] lavc/qsvenc: replace assert with error return

2018-12-07 Thread Carl Eugen Hoyos
2018-12-07 11:14 GMT+01:00, Linjie Fu : > bs->FrameType is not set in MSDK in some cases (mjpeg encode for example), > and assert on a value coming from an external library is not proper. > > Add default type check for bs->FrameType, and return invalid data error in > function > ff_qsv_encode to

[FFmpeg-devel] [PATCH, v3] lavc/qsvenc: replace assert with error return

2018-12-07 Thread Linjie Fu
bs->FrameType is not set in MSDK in some cases (mjpeg encode for example), and assert on a value coming from an external library is not proper. Add default type check for bs->FrameType, and return invalid data error in function ff_qsv_encode to avoid using uninitialized value. Fix #7593.