Bruce Sun [:brsun] Assignee
	Description • 10 years ago

Gecko got crash while parsing / playing Vorbis data in WebM in some cases:
 - https://github.com/mozilla-b2g/platform_frameworks_av/blob/master/media/libstagefright/codecs/vorbis/dec/SoftVorbis.cpp#L274

It seems we have to do extra efforts to handle Vorbis data:
 - https://github.com/mozilla-b2g/platform_frameworks_av/blob/master/media/libstagefright/OMXCodec.cpp#L3128
 - https://github.com/mozilla-b2g/platform_frameworks_av/blob/master/media/libstagefright/NuMediaExtractor.cpp#L472

	Bruce Sun [:brsun] Assignee
	Comment 1 • 10 years ago

One WebM example to make gecko crash after enabling MediaCodecReader:
 - https://bugzilla.mozilla.org/show_bug.cgi?id=823646#c2
 - http://www.quirksmode.org/html5/videos/big_buck_bunny.webm

	Bruce Sun [:brsun] Assignee
	Comment 2 • 10 years ago

Append the queried value of |kKeyValidSamples| to the end of each vorbis input buffer.

	Bruce Sun [:brsun] Assignee
	Comment 3 • 10 years ago

https://tbpl.mozilla.org/?tree=Try&rev=009bafb08174

	Carsten Book [:Tomcat]
	Comment 4 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/039d4e5c03b9

	Carsten Book [:Tomcat]
	Comment 5 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/039d4e5c03b9