RepoMirrors/ffmpeg
1
0
Fork 0
mirror of https://git.ffmpeg.org/ffmpeg.git synced 2024-12-20 06:20:40 +00:00
Code Issues Projects Releases Wiki Activity

avformat/asfcrypt: Fix wrong array length in function declaration

Browse Source 
multiswap_step() and multiswap_inv_step() both only require
six keys; in all current callers, these keys are part of
an array of twelve keys, yet in some of these callers the keys
given to these functions point to the second half of these
twelve keys, so that only six keys are available to these functions.
This led to -Wstringop-overread warnings when compiling with GCC 12.1.
Fix these by adapting the declaration of these functions.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
This commit is contained in:
Andreas Rheinhardt 2022-07-10 20:04:05 +02:00
parent 30c7cff399
commit c807ee6975
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
libavformat/asfcrypt.c
View File

@ -73,7 +73,7 @@ static void multiswap_invert_keys(uint32_t keys[12])
keys[i] = inverse(keys[i]);
}
static uint32_t multiswap_step(const uint32_t keys[12], uint32_t v)
static uint32_t multiswap_step(const uint32_t keys[6], uint32_t v)
{
int i;
v *= keys[0];
@ -85,7 +85,7 @@ static uint32_t multiswap_step(const uint32_t keys[12], uint32_t v)
return v;
}
static uint32_t multiswap_inv_step(const uint32_t keys[12], uint32_t v)
static uint32_t multiswap_inv_step(const uint32_t keys[6], uint32_t v)
{
int i;
v -= keys[5];