[FFmpeg-devel] [PATCH v4] vaapi_encode_mjpeg: fix bad component id bug
andreas.rheinhardt at gmail.com
Thu Jun 13 12:11:00 EEST 2019
Dominik 'Rathann' Mierzejewski:
> On Friday, 07 June 2019 at 23:45, U. Artie Eoff wrote:
>> The compound literals assigned to "components"
>> only exist within the scope of the if/else
>> block (thanks Mark Thompson for the better
>> Thus, after this if/else block, "components"
>> ends up pointing to an arbitrary/undefined
>> array. With some compilers and depending on
>> optimization settings, these arbitrary values
>> may end up being the same value (i.e. 0 with
>> GNU GCC 9.x). Unfortunately, the GNU GCC
>> compiler, at least, never prints any warnings
>> about this.
>> This patch fixes this issue by assigning the
>> constant arrays to local variables at function
>> scope and then pointing "components" to those
>> as necessary.
>> Fixes #7915
> Brilliant detective work, Artie. Could you open a bug report with gcc
> upstream? A case like this should get an explicit warning from gcc like
> you pointed out.
A request for this has already been opened .
It seems that this is also responsible for filesystem corruption on
linux when using a SSD cache + HDD combination (see ).
More information about the ffmpeg-devel