qtdemux: Fix av1C parsing

This is a regression introduced by
https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3882

The av1c codec configuration parsing would always fail due to an off-by-one
error, the content of an atom starting at offset 8 (i.e. the 9th byte) and not
9 (the 10th byte).

Also introduce a break in order to not get stray warnings

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/4433>
This commit is contained in:
Edward Hervey 2023-04-17 09:28:43 +02:00 committed by Edward Hervey
parent 75af296b82
commit 4c6f41a00a

View file

@ -12276,7 +12276,7 @@ qtdemux_parse_trak (GstQTDemux * qtdemux, GNode * trak)
* rest: OBUs.
*/
switch (av1_data[9]) {
switch (av1_data[8]) {
case 0x81:{
guint8 pres_delay_field;
@ -12297,10 +12297,11 @@ qtdemux_parse_trak (GstQTDemux * qtdemux, GNode * trak)
gst_caps_set_simple (entry->caps,
"codec_data", GST_TYPE_BUFFER, buf, NULL);
gst_buffer_unref (buf);
break;
}
default:
GST_WARNING ("Unknown version %d of av1C box",
av1_data[9]);
GST_WARNING ("Unknown version 0x%02x of av1C box",
av1_data[8]);
break;
}