Title: [293726] trunk/Source/WebCore
Revision
293726
Author
[email protected]
Date
2022-05-03 09:00:59 -0700 (Tue, 03 May 2022)

Log 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.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (293725 => 293726)


--- trunk/Source/WebCore/ChangeLog	2022-05-03 15:45:26 UTC (rev 293725)
+++ trunk/Source/WebCore/ChangeLog	2022-05-03 16:00:59 UTC (rev 293726)
@@ -1,3 +1,16 @@
+2022-05-03  Youenn Fablet  <[email protected]>
+
+        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  <[email protected]>
 
         [CSS Cascade Layers] Endless recursion with revert-layer in other tree context

Modified: trunk/Source/WebCore/platform/mediastream/cocoa/AudioMediaStreamTrackRendererUnit.cpp (293725 => 293726)


--- trunk/Source/WebCore/platform/mediastream/cocoa/AudioMediaStreamTrackRendererUnit.cpp	2022-05-03 15:45:26 UTC (rev 293725)
+++ trunk/Source/WebCore/platform/mediastream/cocoa/AudioMediaStreamTrackRendererUnit.cpp	2022-05-03 16:00:59 UTC (rev 293726)
@@ -169,6 +169,7 @@
     if (!m_hasPendingRenderSources)
         return;
 
+    DisableMallocRestrictionsForCurrentThreadScope disableMallocRestrictions;
     m_renderSources = WTFMove(m_pendingRenderSources);
     m_hasPendingRenderSources = false;
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to