1936124 - MP4 with multiple DecoderSpecificInfo entries does not play

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect, P3)

Description

[Alice0775 White]

Attachment: about:support

STR:

1. Open https://bugzilla.mozilla.org/attachment.cgi?id=9442536

Actual results:
No video with supported format and MIME type found.

Error console shows the following error.

Media resource https://bugzilla.mozilla.org/attachment.cgi?id=9442536 could not be decoded. attachment.cgi

Media resource https://bugzilla.mozilla.org/attachment.cgi?id=9442536 could not be decoded, error: Error Code: NS_ERROR_DOM_MEDIA_METADATA_ERR (0x806e0006)
Details: virtual RefPtr<MP4Demuxer::InitPromise> __cdecl mozilla::MP4Demuxer::Init(void): Parse MP4 metadata failed

Expected results:
Video plays. Chrome works fine.

Comment 1

[az [:az] (back 7/20)]

There's a Missing ID_END on the audio stream that shows up in mediainfo:

Audio
ID                                       : 2
Format                                   : AAC LC
Format/Info                              : Advanced Audio Codec Low Complexity
Codec ID                                 : mp4a-40-2
Duration                                 : 7 s 467 ms
Bit rate mode                            : Constant
Bit rate                                 : 96.0 kb/s
Channel(s)                               : 2 channels
Channel layout                           : L R
Sampling rate                            : 48.0 kHz
Frame rate                               : 46.875 FPS (1024 SPF)
Compression mode                         : Lossy
Stream size                              : 87.2 KiB (7%)
Language                                 : English
Encoded date                             : 2024-12-09 19:10:50 UTC
Tagged date                              : 2024-12-09 19:10:50 UTC
Errors                                   : Missing ID_END

Comment 2

[Matthew Gregan [:kinetik]]

This will be addressed by relaxing the requirement that there's a single DecoderSpecificInfo entry (per ISO 14496-1:2010 § 7.2.6.7), except when the parser is in strict mode.

Comment 3

[Matthew Gregan [:kinetik]]

Attachment: GitHub Pull Request

Comment 4

[az [:az] (back 7/20)]

Verified that :kinetik's patch fixes the problem + approved it, let's close this out once we roll the new changes in.

Comment 5

[Mathew Hodson]

Landed in bug 1967030