diff options
author | Martin Storsjo <martin@martin.st> | 2015-06-23 14:51:06 +0300 |
---|---|---|
committer | Marco Nelissen <marcone@google.com> | 2015-06-25 08:25:57 -0700 |
commit | c22addc6ee9c96f2996001362dbe39dd5e14038d (patch) | |
tree | bf7a85e2985bb0c514e733538e96ffdc67f50437 /encoder/irc_fixed_point_error_bits.c | |
parent | 88966cf07d2095aec959c5ac14b30237eeaa5727 (diff) | |
download | android_external_libavc-c22addc6ee9c96f2996001362dbe39dd5e14038d.tar.gz android_external_libavc-c22addc6ee9c96f2996001362dbe39dd5e14038d.tar.bz2 android_external_libavc-c22addc6ee9c96f2996001362dbe39dd5e14038d.zip |
Don't use static structs for rc init
Most of them was benign and didn't risk any data race; they
were used for write-only memory during init.
The array as_itt_memtab in ih264e_get_rate_control_mem_tab
could however cause a real data race, if multiple threads
try to initialize an encoder at the same time, since it
used as temporary storage when mapping between different
structs.
Change-Id: I3ca29f0e3be2fad154becc3ee29867e591975a74
Diffstat (limited to 'encoder/irc_fixed_point_error_bits.c')
-rw-r--r-- | encoder/irc_fixed_point_error_bits.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/encoder/irc_fixed_point_error_bits.c b/encoder/irc_fixed_point_error_bits.c index 42dcfc5..1c35685 100644 --- a/encoder/irc_fixed_point_error_bits.c +++ b/encoder/irc_fixed_point_error_bits.c @@ -62,7 +62,7 @@ WORD32 irc_error_bits_num_fill_use_free_memtab(error_bits_t **pps_error_bits, ITT_FUNC_TYPE_E e_func_type) { WORD32 i4_mem_tab_idx = 0; - static error_bits_t s_error_bits_temp; + error_bits_t s_error_bits_temp; /* * Hack for all alloc, during which we don't have any state memory. |