Title: [165195] trunk/Source/WebCore
Revision
165195
Author
[email protected]
Date
2014-03-06 10:05:20 -0800 (Thu, 06 Mar 2014)

Log Message

[Mac] Leak: dispatch_semaphore leak allocated in MediaPlayerPrivateAVFoundationObjC::waitForVideoOutputMediaDataWillChange
https://bugs.webkit.org/show_bug.cgi?id=129792

Patch by Joseph Pecoraro <[email protected]> on 2014-03-06
Reviewed by Anders Carlsson.

* platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
(WebCore::MediaPlayerPrivateAVFoundationObjC::~MediaPlayerPrivateAVFoundationObjC):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (165194 => 165195)


--- trunk/Source/WebCore/ChangeLog	2014-03-06 17:37:05 UTC (rev 165194)
+++ trunk/Source/WebCore/ChangeLog	2014-03-06 18:05:20 UTC (rev 165195)
@@ -1,3 +1,13 @@
+2014-03-06  Joseph Pecoraro  <[email protected]>
+
+        [Mac] Leak: dispatch_semaphore leak allocated in MediaPlayerPrivateAVFoundationObjC::waitForVideoOutputMediaDataWillChange
+        https://bugs.webkit.org/show_bug.cgi?id=129792
+
+        Reviewed by Anders Carlsson.
+
+        * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
+        (WebCore::MediaPlayerPrivateAVFoundationObjC::~MediaPlayerPrivateAVFoundationObjC):
+
 2014-03-06  Brendan Long  <[email protected]>
 
         Implement VideoTrackList.selectedIndex

Modified: trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm (165194 => 165195)


--- trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm	2014-03-06 17:37:05 UTC (rev 165194)
+++ trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm	2014-03-06 18:05:20 UTC (rev 165195)
@@ -351,6 +351,8 @@
 #if HAVE(AVFOUNDATION_VIDEO_OUTPUT)
     [m_videoOutputDelegate setCallback:0];
     [m_videoOutput setDelegate:nil queue:0];
+    if (m_videoOutputSemaphore)
+        dispatch_release(m_videoOutputSemaphore);
 #endif
     cancelLoad();
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to