flodavid/suyu
1
0
Fork 0
forked from suyu/suyu
Code Pull requests Activity

vp9: Provide a default initializer for "hidden" member

Browse source 
The API of VP9 exposes a WasFrameHidden() function which accesses this
member. Given the constructor previously didn't initialize this member,
it's a potential vector for an uninitialized read.

Instead, we can initialize this to a deterministic value to prevent that
from occurring.
This commit is contained in:
Lioncash 2020-10-29 22:35:52 -04:00
parent f8543249f0
commit badea3b301
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/video_core/command_classes/codecs/vp9.h
View file

@ -159,7 +159,7 @@ private:
std::array<s8, 4> loop_filter_ref_deltas{}; std::array<s8, 4> loop_filter_ref_deltas{};
std::array<s8, 2> loop_filter_mode_deltas{}; std::array<s8, 2> loop_filter_mode_deltas{};
bool hidden; bool hidden = false;
s64 current_frame_number = -2; // since we buffer 2 frames s64 current_frame_number = -2; // since we buffer 2 frames
s32 grace_period = 6; // frame offsets need to stabilize s32 grace_period = 6; // frame offsets need to stabilize
std::array<FrameContexts, 4> frame_ctxs{}; std::array<FrameContexts, 4> frame_ctxs{};