Changeset 239419 in webkit

Timestamp:

Dec 19, 2018, 7:24:28 PM (8 years ago)

Author:

jer.noble@apple.com

Message:

Leak of MTAudioProcessingTap (304 bytes) in com.apple.WebKit.WebContent running WebKit layout tests
​https://bugs.webkit.org/show_bug.cgi?id=192896
<rdar://46732186>

Reviewed by Eric Carlson.

• platform/graphics/avfoundation/AudioSourceProviderAVFObjC.mm:

(WebCore::AudioSourceProviderAVFObjC::initCallback):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/avfoundation/AudioSourceProviderAVFObjC.mm (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2018-12-19  Jer Noble  <jer.noble@apple.com>
+
+        Leak of MTAudioProcessingTap (304 bytes) in com.apple.WebKit.WebContent running WebKit layout tests
+        https://bugs.webkit.org/show_bug.cgi?id=192896
+        <rdar://46732186>
+
+        Reviewed by Eric Carlson.
+
+        * platform/graphics/avfoundation/AudioSourceProviderAVFObjC.mm:
+        (WebCore::AudioSourceProviderAVFObjC::initCallback):
+
 2018-12-19  Timothy Hatcher  <timothy@apple.com>
 

trunk/Source/WebCore/platform/graphics/avfoundation/AudioSourceProviderAVFObjC.mm

@@ -238 +238 @@
     ASSERT(tap);
     AudioSourceProviderAVFObjC* _this = static_cast<AudioSourceProviderAVFObjC*>(clientInfo);
-    _this->m_tap = tap;
+    _this->m_tap = adoptCF(tap);
     _this->m_tapStorage = new TapStorage(_this);
     _this->init(clientInfo, tapStorageOut);