Bug 26081 - Reproducible crash in TimeRanges::copy() at openvideo.dailymotion.com
Summary: Reproducible crash in TimeRanges::copy() at openvideo.dailymotion.com
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P1 Normal
Assignee: Nobody
URL: http://openvideo.dailymotion.com/de/f...
Keywords: InRadar
: 26261 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-05-29 10:25 PDT by lars.sonchocky-helldorf
Modified: 2009-06-08 16:12 PDT (History)
3 users (show)

See Also:


Attachments
crashlog for bug 26081 (32.12 KB, text/plain)
2009-05-29 10:29 PDT, lars.sonchocky-helldorf
no flags Details
crash log for 10.4.11 PPC (27.25 KB, text/plain)
2009-05-29 11:08 PDT, lars.sonchocky-helldorf
no flags Details
patch v1. (3.00 KB, patch)
2009-06-02 12:07 PDT, Pierre d'Herbemont
darin: review+
Details | Formatted Diff | Diff
patch 2. (3.28 KB, patch)
2009-06-02 16:47 PDT, Pierre d'Herbemont
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description lars.sonchocky-helldorf 2009-05-29 10:25:53 PDT
I get a crash for r44201 at http://openvideo.dailymotion.com/de/featured/channel/news/video/x9e0tu_euromaxx-the-truth-about-germany_people and all the other videos on http://openvideo.dailymotion.com/ (subpages)

see attached crashlog
Comment 1 lars.sonchocky-helldorf 2009-05-29 10:29:57 PDT
Created attachment 30779 [details]
crashlog for bug 26081

crashlog attached
Comment 2 lars.sonchocky-helldorf 2009-05-29 10:34:45 PDT
o.k. http://openvideo.dailymotion.com/de/channel/sport/featured seems to work
Comment 3 lars.sonchocky-helldorf 2009-05-29 11:08:24 PDT
Created attachment 30780 [details]
crash log for 10.4.11 PPC

another crash log for PPC on Tiger
Comment 4 Alexey Proskuryakov 2009-05-30 02:35:13 PDT
<rdar://problem/6934752>
Comment 5 Pierre d'Herbemont 2009-06-02 12:07:31 PDT
Created attachment 30879 [details]
patch v1.
Comment 6 Pierre d'Herbemont 2009-06-02 16:47:54 PDT
Created attachment 30886 [details]
patch 2.

This time with a layout test that is a real HTML document.
Comment 7 Eric Carlson 2009-06-04 10:00:50 PDT
http://trac.webkit.org/changeset/44415
Comment 8 lars.sonchocky-helldorf 2009-06-05 10:11:14 PDT
I am missing the file video-test.js (which is referenced in before-load-member-access.html) in your commit. Without that file I just get a 

ReferenceError: Can't find variable: findMediaElement

when I open before-load-member-access.html with r44410 but no crash (this version should still crash since the crash was fixed not before r44415 -shouldn't it?)

I think the testcase is "toothless" without video-test.js

btw. it looks like dailymotion fixed/changed their code since I no longer get a crash even with r44410
Comment 9 Eric Carlson 2009-06-05 10:26:08 PDT
video-tests.js is not new to this check-in, it is used by most of the media layout tests: http://trac.webkit.org/browser/trunk/LayoutTests/media/video-test.js
Comment 10 lars.sonchocky-helldorf 2009-06-05 16:29:55 PDT
(In reply to comment #9)
> video-tests.js is not new to this check-in, it is used by most of the media
> layout tests:
> http://trac.webkit.org/browser/trunk/LayoutTests/media/video-test.js
> 

Sorry for the turmoil, I wasn't aware of this. But now I can confirm that the bug is fixed as of r44438 for both PPC and Intel (after running the testcase manually).
Comment 11 Mark Rowe (bdash) 2009-06-08 16:12:07 PDT
*** Bug 26261 has been marked as a duplicate of this bug. ***