diff options
| author | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2021-09-13 14:38:50 +0200 |
|---|---|---|
| committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2022-01-11 22:37:58 +0100 |
| commit | edaf8c961510ec7a4d83fdbd4b0986ae67ee1357 (patch) | |
| tree | 0719d7cd18d376920a7792e995fcdcb87c476179 | |
| parent | 953ad7b362d6bd05669ad063cfa11e13b12c8362 (diff) | |
| download | ffmpeg-edaf8c961510ec7a4d83fdbd4b0986ae67ee1357.tar.gz | |
avcodec/opus_silk: Remove wrong size information in function declaration
silk_lsp2poly()'s declaration contained arrays with array sizes;
yet these array sizes exceeded the number of actually accessed array
elements (which is related to another parameter) and this leads to
-Wstringop-overflow= warnings from GCC 11, because the arrays provided
by callers are only large enough for the actually used elements.
So replace the incorrect array sizes with comments containing
the correct array sizes. Given that these sizes are not compile-time
constants, they can only be communicated via a comment.
Reported by Paul B Mahol.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
(cherry picked from commit 4b154743163ffbe3fdc50759c0c55dc854636488)
| -rw-r--r-- | libavcodec/opus_silk.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/libavcodec/opus_silk.c b/libavcodec/opus_silk.c index 913053c5e2..8523b55ada 100644 --- a/libavcodec/opus_silk.c +++ b/libavcodec/opus_silk.c @@ -198,7 +198,8 @@ static inline int silk_is_lpc_stable(const int16_t lpc[16], int order) } } -static void silk_lsp2poly(const int32_t lsp[16], int32_t pol[16], int half_order) +static void silk_lsp2poly(const int32_t lsp[/* 2 * half_order - 1 */], + int32_t pol[/* half_order + 1 */], int half_order) { int i, j; |