Title: [214699] trunk/Source/WebCore
Revision
214699
Author
[email protected]
Date
2017-03-31 16:55:07 -0700 (Fri, 31 Mar 2017)

Log Message

Fix memory leak in RealtimeVideoIncomingSource
https://bugs.webkit.org/show_bug.cgi?id=170356

Patch by Youenn Fablet <[email protected]> on 2017-03-31
Reviewed by Eric Carlson.

No change of behavior.

* platform/mediastream/mac/RealtimeIncomingVideoSource.cpp:
(WebCore::RealtimeIncomingVideoSource::OnFrame): Adopting the newly created pointer.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (214698 => 214699)


--- trunk/Source/WebCore/ChangeLog	2017-03-31 23:38:17 UTC (rev 214698)
+++ trunk/Source/WebCore/ChangeLog	2017-03-31 23:55:07 UTC (rev 214699)
@@ -1,3 +1,15 @@
+2017-03-31  Youenn Fablet  <[email protected]>
+
+        Fix memory leak in RealtimeVideoIncomingSource
+        https://bugs.webkit.org/show_bug.cgi?id=170356
+
+        Reviewed by Eric Carlson.
+
+        No change of behavior.
+
+        * platform/mediastream/mac/RealtimeIncomingVideoSource.cpp:
+        (WebCore::RealtimeIncomingVideoSource::OnFrame): Adopting the newly created pointer.
+
 2017-03-31  Andy Estes  <[email protected]>
 
         REGRESSION (r202472): Data Detection overwrites existing links in detected ranges

Modified: trunk/Source/WebCore/platform/mediastream/mac/RealtimeIncomingVideoSource.cpp (214698 => 214699)


--- trunk/Source/WebCore/platform/mediastream/mac/RealtimeIncomingVideoSource.cpp	2017-03-31 23:38:17 UTC (rev 214698)
+++ trunk/Source/WebCore/platform/mediastream/mac/RealtimeIncomingVideoSource.cpp	2017-03-31 23:55:07 UTC (rev 214699)
@@ -160,7 +160,7 @@
     }
     CFRelease(formatDescription);
 
-    RetainPtr<CMSampleBufferRef> sample = sampleBuffer;
+    auto sample = adoptCF(sampleBuffer);
 
     CFArrayRef attachmentsArray = CMSampleBufferGetSampleAttachmentsArray(sampleBuffer, true);
     for (CFIndex i = 0; i < CFArrayGetCount(attachmentsArray); ++i) {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to