]> git.sesse.net Git - ffmpeg/commit
vaapi_encode_mjpeg: fix bad component id bug
authorU. Artie Eoff <ullysses.a.eoff@intel.com>
Fri, 7 Jun 2019 21:45:33 +0000 (14:45 -0700)
committerMark Thompson <sw@jkqxz.net>
Wed, 12 Jun 2019 21:27:13 +0000 (22:27 +0100)
commitf70c397456c7eba78a22e9318bce634e1d9079d6
tree7f219160fb61d30e72cac0a4aeba321cd05c38ce
parentb8f1542dcbadd25b027328acf4f312e1780499a5
vaapi_encode_mjpeg: fix bad component id bug

The compound literals assigned to "components"
only exist within the scope of the if/else
block (thanks Mark Thompson for the better
explanation).

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

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
libavcodec/vaapi_encode_mjpeg.c