Bug 26081

Summary:

Reproducible crash in TimeRanges::copy() at openvideo.dailymotion.com

Product:

WebKit

Reporter:

lars.sonchocky-helldorf

Component:

WebCore Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

eric.carlson, hartman.wiki, simon.fraser

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

URL:

http://openvideo.dailymotion.com/de/featured/channel/news/video/x9e0tu_euromaxx-the-truth-about-germany_people

Attachments:

Description	Flags
crashlog for bug 26081	none
crash log for 10.4.11 PPC	none
patch v1.	darin: review+
patch 2.	darin: review+

lars.sonchocky-helldorf

Reported 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

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
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

lars.sonchocky-helldorf

Comment 1 2009-05-29 10:29:57 PDT

Created attachment 30779 [details] crashlog for bug 26081 crashlog attached

lars.sonchocky-helldorf

Comment 2 2009-05-29 10:34:45 PDT

o.k. http://openvideo.dailymotion.com/de/channel/sport/featured seems to work

lars.sonchocky-helldorf

Comment 3 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

Alexey Proskuryakov

Comment 4 2009-05-30 02:35:13 PDT

<rdar://problem/6934752>

Pierre d'Herbemont

Comment 5 2009-06-02 12:07:31 PDT

Created attachment 30879 [details] patch v1.

Pierre d'Herbemont

Comment 6 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.

Eric Carlson

Comment 7 2009-06-04 10:00:50 PDT

http://trac.webkit.org/changeset/44415

lars.sonchocky-helldorf

Comment 8 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

Eric Carlson

Comment 9 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

lars.sonchocky-helldorf

Comment 10 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).

Mark Rowe (bdash)

Comment 11 2009-06-08 16:12:07 PDT

*** Bug 26261 has been marked as a duplicate of this bug. ***

Note You need to log in before you can comment on or make changes to this bug.