SVN r16552 includes a fix to libvorbis that prevents the decoder from rejecting some valid files produced with the beta4 encoder. These may not be particularly common files, but we should pick this simple fix up anyway.
Created attachment 402938 [details] [diff] [review] patch v0 Update libvorbis to SVN r16552. This needs a test, but I don't have a file which is fixed by this patch handy.
It's actually the beta1 encoder. Test file in URL. Unfortunately it doesn't play on trunk *or* with the patch applied. Haven't investigated why yet.
Summary: Update libvorbis to pick up fix for some files encoded with libvorbis beta4 → Update libvorbis to pick up fix for some files encoded with libvorbis beta1
Flags: wanted1.9.2? → wanted1.9.2+
Created attachment 404185 [details] [diff] [review] patch v1 Same, but with a test. This also fixes the comments above gPlayTests, which should've been above gErrorTests (I messed them up in bug 481057). Included test file does not play on current trunk, but does play with patch applied (comment 2 is out of date--suspect that was fixed by oggplay updates). Note that the test does not fail every time when the patch is not applied, but I can't work out why.
Attachment #402938 - Attachment is obsolete: true
We took the broken libvorbis update on 1.9.1 for 220.127.116.11, so we should take this fix to ensure we don't regress anything. Also see bug 507167 comment 16.
status1.9.1: --- → ?
Posted the libvorbis update to bug 507167. Includes the test from this bug. Marking this as depending on that.
Depends on: 507167
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Attachment #404185 - Attachment is obsolete: true
Fixed on 1.9.2 by checkin of bug 507167.
status1.9.2: --- → beta1-fixed
Checked in to 1.9.1 too.
status1.9.1: ? → .4-fixed
Do we have sample rejected files somewhere that can be used to verify this fix for Firefox 3.5.4?
There's one in the bug URL. I don't have any others to test with.
Er, I missed the url. Sorry about that. I verified that it doesn't play in 18.104.22.168 on OS X but does play on 22.214.171.124 (Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:126.96.36.199) Gecko/20091016 Firefox/3.5.4) so this looks good.
You need to log in before you can comment on or make changes to this bug.