Title: [201403] trunk/Source/WebCore
- Revision
- 201403
- Author
- jer.no...@apple.com
- Date
- 2016-05-25 14:57:03 -0700 (Wed, 25 May 2016)
Log Message
CRASH at WebCore::WebPlaybackSessionModelMediaElement::selectAudioMediaOption() + 104
https://bugs.webkit.org/show_bug.cgi?id=158090
<rdar://problem/26388936>
Reviewed by Eric Carlson.
Null-check m_mediaElement before using.
* platform/cocoa/WebPlaybackSessionModelMediaElement.mm:
(WebPlaybackSessionModelMediaElement::selectAudioMediaOption):
(WebPlaybackSessionModelMediaElement::selectLegibleMediaOption):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (201402 => 201403)
--- trunk/Source/WebCore/ChangeLog 2016-05-25 21:19:09 UTC (rev 201402)
+++ trunk/Source/WebCore/ChangeLog 2016-05-25 21:57:03 UTC (rev 201403)
@@ -1,3 +1,17 @@
+2016-05-25 Jer Noble <jer.no...@apple.com>
+
+ CRASH at WebCore::WebPlaybackSessionModelMediaElement::selectAudioMediaOption() + 104
+ https://bugs.webkit.org/show_bug.cgi?id=158090
+ <rdar://problem/26388936>
+
+ Reviewed by Eric Carlson.
+
+ Null-check m_mediaElement before using.
+
+ * platform/cocoa/WebPlaybackSessionModelMediaElement.mm:
+ (WebPlaybackSessionModelMediaElement::selectAudioMediaOption):
+ (WebPlaybackSessionModelMediaElement::selectLegibleMediaOption):
+
2016-05-25 Brady Eidson <beid...@apple.com>
Race condition calling back to an IDBOpenDBRequest during WorkerThread shutdown.
Modified: trunk/Source/WebCore/platform/cocoa/WebPlaybackSessionModelMediaElement.mm (201402 => 201403)
--- trunk/Source/WebCore/platform/cocoa/WebPlaybackSessionModelMediaElement.mm 2016-05-25 21:19:09 UTC (rev 201402)
+++ trunk/Source/WebCore/platform/cocoa/WebPlaybackSessionModelMediaElement.mm 2016-05-25 21:57:03 UTC (rev 201403)
@@ -229,6 +229,9 @@
void WebPlaybackSessionModelMediaElement::selectAudioMediaOption(uint64_t selectedAudioIndex)
{
+ if (!m_mediaElement)
+ return;
+
ASSERT(selectedAudioIndex < std::numeric_limits<size_t>::max());
AudioTrack* selectedAudioTrack = nullptr;
@@ -244,6 +247,9 @@
void WebPlaybackSessionModelMediaElement::selectLegibleMediaOption(uint64_t index)
{
+ if (!m_mediaElement)
+ return;
+
ASSERT(index < std::numeric_limits<size_t>::max());
TextTrack* textTrack = nullptr;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes