On 19/03/16 08:47, Anton Khirnov wrote:
> Quoting Mark Thompson (2016-03-16 00:00:09)
>> +
>> +static int scale_vaapi_pipeline_uninit(ScaleVAAPIContext *ctx)
>> +{
>> +    if (ctx->va_context != VA_INVALID_ID) {
> 
> Some tests reveal that these checks are not good, because this function
> can be called without the context/config ids ever being initialized.
> E.g. if the caller provides invalid filter options, init is never
> called, but uninit is.
> 

Oh, I wasn't expecting uninit to ever be called without init.  So just add an 
extra "init called" field to check there?

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to