Bug 242453

Summary:	Crash under MediaPlayerPrivateRemote::durationMediaTime()
Product:	WebKit	Reporter:	Chris Dumez <cdumez>
Component:	Media	Assignee:	Chris Dumez <cdumez>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Chris Dumez

Reported 2022-07-07 09:52:19 PDT

Crash under MediaPlayerPrivateRemote::durationMediaTime(): Thread 0 Crashed:: Dispatch queue: com.apple.main-thread 0 com.apple.WebKit 0x1b56b0eb0 WebKit::MediaPlayerPrivateRemote::durationMediaTime() const + 16 (/Sources/WebKit/Source/WebKit/WebProcess/GPU/media/MediaSourcePrivateRemote.h:77) 1 com.apple.WebCore 0x1b247bfcc WebCore::HTMLMediaElement::endedPlayback() const + 32 (Sources/WebCore/Source/WebCore/./html/HTMLMediaElement.cpp:5553) 2 com.apple.WebCore 0x1b386fe08 WebCore::HTMLMediaElement::mediaState() const + 420 (Source/WebCore/./html/HTMLMediaElement.cpp:8204) 3 com.apple.WebCore 0x1b35d7380 WebCore::Document::updateIsPlayingMedia() + 140 (Source/WebCore/./dom/Document.cpp:4475) 4 com.apple.WebCore 0x1b35d7230 WebCore::Document::addAudioProducer(WebCore::MediaProducer&) + 48 (Sources/WebCore/Source/WebCore/./dom/Document.cpp:4439) 5 com.apple.WebCore 0x1b38598e4 WebCore::HTMLMediaElement::initializeMediaSession() + 432 (Source/WebCore/./html/HTMLMediaElement.cpp:534) 6 com.apple.WebCore 0x1b385b3c0 WebCore::HTMLMediaElement::didFinishInsertingNode() + 272 (Source/WebCore/./html/HTMLMediaElement.cpp:8569) 7 com.apple.WebCore 0x1b35a620c WebCore::ContainerNode::appendChildWithoutPreInsertionValidityCheck(WebCore::Node&) + 796 (Sources/WebCore/Source/WebCore/./dom/ContainerNode.cpp:274)

Attachments
Add attachment proposed patch, testcase, etc.

Chris Dumez

Comment 1 2022-07-07 09:52:29 PDT

<rdar://96267301>

Chris Dumez

Comment 2 2022-07-07 09:57:08 PDT

Pull request: https://github.com/WebKit/WebKit/pull/2173

EWS

Comment 3 2022-07-07 10:56:54 PDT

Committed 252226@main (96f399638f0c): <https://commits.webkit.org/252226@main> Reviewed commits have been landed. Closing PR #2173 and removing active labels.

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