diff options
author | U. Artie Eoff <ullysses.a.eoff@intel.com> | 2019-06-07 14:45:33 -0700 |
---|---|---|
committer | Mark Thompson <sw@jkqxz.net> | 2019-06-12 22:27:13 +0100 |
commit | f70c397456c7eba78a22e9318bce634e1d9079d6 (patch) | |
tree | 7f219160fb61d30e72cac0a4aeba321cd05c38ce /tests/ref/lavf | |
parent | b8f1542dcbadd25b027328acf4f312e1780499a5 (diff) | |
download | ffmpeg-f70c397456c7eba78a22e9318bce634e1d9079d6.tar.gz |
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>
Diffstat (limited to 'tests/ref/lavf')
0 files changed, 0 insertions, 0 deletions