While working in color balance interface in vaapipostroc , I ran across with an assertion that crashed the execution:
gen75_picture_process.c:169: VAStatus gen75_proc_picture(VADriverContextP, VAProfile, union codec_state *, struct hw_context *): Assertion `pipeline_param->num_filters <= 4' failed.
In my opinion, this shouldn't be handled in this way. If the hardware offers only a limited number of filters to activate simultaneously, it should behave different (ignore the new filter, perhaps) but it shouldn't crash.
whether return error is the better idea if pipeline_param->num_filters <= 4 is not true? it makes the user confused if the driver ignores the new filters at this case.