diff mbox series

[FFmpeg-devel,02/10] avformat/aadec: Don't use the same loop counter in inner and outer loop

Message ID AM7PR03MB666026E673177788B390C9548F6D9@AM7PR03MB6660.eurprd03.prod.outlook.com
State Accepted
Commit 0a76f8217e0d80d336ad8d9c46523fd3d1262c8c
Headers show
Series [FFmpeg-devel,01/10] avformat/utils: Make ff_data_to_hex() zero-terminate the string | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc success Make fate finished

Commit Message

Andreas Rheinhardt Dec. 6, 2021, 1:12 a.m. UTC 
Due to this bush.aa (from the FATE suite) exported garbage metadata
with key "_040930".

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavformat/aadec.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/libavformat/aadec.c b/libavformat/aadec.c
index 72f4b6f7cc..a3a7664469 100644
--- a/libavformat/aadec.c
+++ b/libavformat/aadec.c
@@ -132,8 +132,8 @@  static int aa_read_header(AVFormatContext *s)
                 AV_WB32(&header_key[idx * 4], header_key_part[idx]); // convert each part to BE!
             }
             av_log(s, AV_LOG_DEBUG, "Processed HeaderKey is ");
-            for (i = 0; i < 16; i++)
-                av_log(s, AV_LOG_DEBUG, "%02x", header_key[i]);
+            for (int j = 0; j < 16; j++)
+                av_log(s, AV_LOG_DEBUG, "%02x", header_key[j]);
             av_log(s, AV_LOG_DEBUG, "\n");
         } else {
             av_dict_set(&s->metadata, key, val, 0);