RESOLVED FIXED 94950
Fix a crash in SourceBufferList.remove().
https://bugs.webkit.org/show_bug.cgi?id=94950
Summary Fix a crash in SourceBufferList.remove().
Aaron Colwell
Reported Friday, August 24, 2012 6:30:46 PM UTC
Fix a crash in SourceBufferList.remove().
Attachments
Patch (1.82 KB, patch)
2012-08-24 10:40 PDT, Aaron Colwell
no flags
Patch (5.11 KB, patch)
2012-08-30 09:04 PDT, Aaron Colwell
no flags
Aaron Colwell
Comment 1 Friday, August 24, 2012 6:40:36 PM UTC
Kentaro Hara
Comment 2 Saturday, August 25, 2012 1:43:17 AM UTC
Could you add a test for your change?
Eric Seidel (no email)
Comment 3 Monday, August 27, 2012 9:23:13 PM UTC
Comment on attachment 160448 [details] Patch We'll need a test if possible. (Or an explanation in the ChangeLog why testing is impossible/iimpractical.)
Aaron Colwell
Comment 4 Tuesday, August 28, 2012 7:16:03 PM UTC
Closing in favor of https://bugs.webkit.org/show_bug.cgi?id=95217 Attempting to create a test for this bug exposed a larger problem with the code.
Aaron Colwell
Comment 5 Tuesday, August 28, 2012 9:18:33 PM UTC
Reopening. eric.carlson@ asked me to keep the fix for this separate from https://bugs.webkit.org/show_bug.cgi?id=95217
Aaron Colwell
Comment 6 Thursday, August 30, 2012 5:04:13 PM UTC
WebKit Review Bot
Comment 7 Thursday, August 30, 2012 5:51:55 PM UTC
Comment on attachment 161486 [details] Patch Clearing flags on attachment: 161486 Committed r127153: <http://trac.webkit.org/changeset/127153>
WebKit Review Bot
Comment 8 Thursday, August 30, 2012 5:51:58 PM UTC
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.