mp3utils: zero8 was only 2 bytes, but 8 are dereferenced, true to the name.

icculus · icculus · commit 204751967a93 · 2025-04-04T11:05:00.000-04:00
diff --git a/src/codecs/mp3utils.c b/src/codecs/mp3utils.c
@@ -665,7 +665,7 @@ static bool parse_ape(Mix_MusicMetaTags *out_tags, struct mp3file_t *src, Sint64
 {
     Uint8 buffer[APE_BUFFER_SIZE + 1];
     Uint32 v, i, tag_size, tag_items_count, tag_item_size;
-    Uint32 zero8[2] = {0, 0};
+    Uint32 zero8[8] = { 0, 0, 0, 0, 0, 0, 0, 0 };
     Sint64 file_size, cur_tag;
     size_t read_size;
 

Original file line number	Diff line number	Diff line change
`@@ -665,7 +665,7 @@ static bool parse_ape(Mix_MusicMetaTags out_tags, struct mp3file_t src, Sint64`
`665`	`665`	`{`
`666`	`666`	`Uint8 buffer[APE_BUFFER_SIZE + 1];`
`667`	`667`	`Uint32 v, i, tag_size, tag_items_count, tag_item_size;`
`668`		`- Uint32 zero8[2] = {0, 0};`
	`668`	`+ Uint32 zero8[8] = { 0, 0, 0, 0, 0, 0, 0, 0 };`
`669`	`669`	`Sint64 file_size, cur_tag;`
`670`	`670`	`size_t read_size;`
`671`	`671`