Changeset 293726 in webkit

Timestamp:

May 3, 2022, 9:00:59 AM (4 years ago)

Author:

youenn@apple.com

Message:

AudioMediaStreamTrackRendererUnit::updateRenderSourcesIfNecessary can free memory
​https://bugs.webkit.org/show_bug.cgi?id=240002

Reviewed by Eric Carlson.

Manually tested.

• platform/mediastream/cocoa/AudioMediaStreamTrackRendererUnit.cpp:

Overwriting the source vector may lead to freeing AudioSampleDataSources.
This is ok as this happens rarely, add DisableMallocRestrictionsForCurrentThreadScope for that purpose.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/mediastream/cocoa/AudioMediaStreamTrackRendererUnit.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2022-05-03  Youenn Fablet  <youenn@apple.com>
+
+        AudioMediaStreamTrackRendererUnit::updateRenderSourcesIfNecessary can free memory
+        https://bugs.webkit.org/show_bug.cgi?id=240002
+
+        Reviewed by Eric Carlson.
+
+        Manually tested.
+
+        * platform/mediastream/cocoa/AudioMediaStreamTrackRendererUnit.cpp:
+        Overwriting the source vector may lead to freeing AudioSampleDataSources.
+        This is ok as this happens rarely, add DisableMallocRestrictionsForCurrentThreadScope for that purpose.
+
 2022-05-03  Antti Koivisto  <antti@apple.com>
 

trunk/Source/WebCore/platform/mediastream/cocoa/AudioMediaStreamTrackRendererUnit.cpp

@@ -170 +170 @@
         return;
 
+    DisableMallocRestrictionsForCurrentThreadScope disableMallocRestrictions;
     m_renderSources = WTFMove(m_pendingRenderSources);
     m_hasPendingRenderSources = false;