Attachment 341679 Details for Bug 186152 – Patch v1

[patch] Patch v1

bug-186152-20180531113404.patch (text/plain), 5.57 KB, created by David Kilzer (:ddkilzer) on 2018-05-31 11:34:05 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: David Kilzer (:ddkilzer)

Created: 2018-05-31 11:34:05 PDT

Size: 5.57 KB

patch

obsolete

>Subversion Revision: 232304
>diff --git a/Source/ThirdParty/libwebrtc/ChangeLog b/Source/ThirdParty/libwebrtc/ChangeLog
>index 26c7f5db745fb8b8ad071bef930667ce4c29887c..08472086ab5db44e66c1d078893c1e757dccbab1 100644
>--- a/Source/ThirdParty/libwebrtc/ChangeLog
>+++ b/Source/ThirdParty/libwebrtc/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-05-31  David Kilzer  <ddkilzer@apple.com>
>+
>+        Fix leak of AudioDeviceID array due to an early return in AudioDeviceMac::GetNumberDevices()
>+        <https://webkit.org/b/186152>
>+        <rdar://problem/40692824>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Source/webrtc/modules/audio_device/mac/audio_device_mac.cc:
>+        Use std::unique_ptr<> so that memory is freed when the pointer
>+        goes out of scope.  Clean up manual calls to free().
>+        * WebKit/0011-Fix-AudioDeviceID-array-leak.patch: Add.
>+
> 2018-05-30  David Kilzer  <ddkilzer@apple.com>
> 
>         Fix leak of a CVPixelBufferRef due to early rerturn in -[RTCVideoEncoderH264 encode:codecSpecificInfo:frameTypes:]
>diff --git a/Source/ThirdParty/libwebrtc/Source/webrtc/modules/audio_device/mac/audio_device_mac.cc b/Source/ThirdParty/libwebrtc/Source/webrtc/modules/audio_device/mac/audio_device_mac.cc
>index c585c32cafc30f40c2dd00bc909649876da20d6a..7b9d1fee29e155c8342b712b5201dd9e87592fde 100644
>--- a/Source/ThirdParty/libwebrtc/Source/webrtc/modules/audio_device/mac/audio_device_mac.cc
>+++ b/Source/ThirdParty/libwebrtc/Source/webrtc/modules/audio_device/mac/audio_device_mac.cc
>@@ -19,6 +19,7 @@
> #include <ApplicationServices/ApplicationServices.h>
> #include <libkern/OSAtomic.h>  // OSAtomicCompareAndSwap()
> #include <mach/mach.h>         // mach_task_self()
>+#include <memory>              // std::unique_ptr<>()
> #include <sys/sysctl.h>        // sysctlbyname()
> 
> namespace webrtc {
>@@ -1564,7 +1565,7 @@ int32_t AudioDeviceMac::GetNumberDevices(const AudioObjectPropertyScope scope,
>     return 0;
>   }
> 
>-  AudioDeviceID* deviceIds = (AudioDeviceID*)malloc(size);
>+  std::unique_ptr<AudioDeviceID[], decltype(free)*> deviceIds(reinterpret_cast<AudioDeviceID*>(malloc(size)), free);
>   UInt32 numberDevices = size / sizeof(AudioDeviceID);
>   AudioBufferList* bufferList = NULL;
>   UInt32 numberScopedDevices = 0;
>@@ -1597,7 +1598,7 @@ int32_t AudioDeviceMac::GetNumberDevices(const AudioObjectPropertyScope scope,
>   bool listOK = true;
> 
>   WEBRTC_CA_LOG_ERR(AudioObjectGetPropertyData(
>-      kAudioObjectSystemObject, &propertyAddress, 0, NULL, &size, deviceIds));
>+      kAudioObjectSystemObject, &propertyAddress, 0, NULL, &size, deviceIds.get()));
>   if (err != noErr) {
>     listOK = false;
>   } else {
>@@ -1641,11 +1642,6 @@ int32_t AudioDeviceMac::GetNumberDevices(const AudioObjectPropertyScope scope,
>   }
> 
>   if (!listOK) {
>-    if (deviceIds) {
>-      free(deviceIds);
>-      deviceIds = NULL;
>-    }
>-
>     if (bufferList) {
>       free(bufferList);
>       bufferList = NULL;
>@@ -1654,12 +1650,6 @@ int32_t AudioDeviceMac::GetNumberDevices(const AudioObjectPropertyScope scope,
>     return -1;
>   }
> 
>-  // Happy ending
>-  if (deviceIds) {
>-    free(deviceIds);
>-    deviceIds = NULL;
>-  }
>-
>   return numberScopedDevices;
> }
> 
>diff --git a/Source/ThirdParty/libwebrtc/WebKit/0011-Fix-AudioDeviceID-array-leak.patch b/Source/ThirdParty/libwebrtc/WebKit/0011-Fix-AudioDeviceID-array-leak.patch
>new file mode 100644
>index 0000000000000000000000000000000000000000..e1d49fbf1abbaee583db1d1c2ec160d23f2242e1
>--- /dev/null
>+++ b/Source/ThirdParty/libwebrtc/WebKit/0011-Fix-AudioDeviceID-array-leak.patch
>@@ -0,0 +1,55 @@
>+diff --git a/Source/ThirdParty/libwebrtc/Source/webrtc/modules/audio_device/mac/audio_device_mac.cc b/Source/ThirdParty/libwebrtc/Source/webrtc/modules/audio_device/mac/audio_device_mac.cc
>+index c585c32cafc..7b9d1fee29e 100644
>+--- a/Source/ThirdParty/libwebrtc/Source/webrtc/modules/audio_device/mac/audio_device_mac.cc
>++++ b/Source/ThirdParty/libwebrtc/Source/webrtc/modules/audio_device/mac/audio_device_mac.cc
>+@@ -19,6 +19,7 @@
>+ #include <ApplicationServices/ApplicationServices.h>
>+ #include <libkern/OSAtomic.h>  // OSAtomicCompareAndSwap()
>+ #include <mach/mach.h>         // mach_task_self()
>++#include <memory>              // std::unique_ptr<>()
>+ #include <sys/sysctl.h>        // sysctlbyname()
>+ 
>+ namespace webrtc {
>+@@ -1564,7 +1565,7 @@ int32_t AudioDeviceMac::GetNumberDevices(const AudioObjectPropertyScope scope,
>+     return 0;
>+   }
>+ 
>+-  AudioDeviceID* deviceIds = (AudioDeviceID*)malloc(size);
>++  std::unique_ptr<AudioDeviceID[], decltype(free)*> deviceIds(reinterpret_cast<AudioDeviceID*>(malloc(size)), free);
>+   UInt32 numberDevices = size / sizeof(AudioDeviceID);
>+   AudioBufferList* bufferList = NULL;
>+   UInt32 numberScopedDevices = 0;
>+@@ -1597,7 +1598,7 @@ int32_t AudioDeviceMac::GetNumberDevices(const AudioObjectPropertyScope scope,
>+   bool listOK = true;
>+ 
>+   WEBRTC_CA_LOG_ERR(AudioObjectGetPropertyData(
>+-      kAudioObjectSystemObject, &propertyAddress, 0, NULL, &size, deviceIds));
>++      kAudioObjectSystemObject, &propertyAddress, 0, NULL, &size, deviceIds.get()));
>+   if (err != noErr) {
>+     listOK = false;
>+   } else {
>+@@ -1641,11 +1642,6 @@ int32_t AudioDeviceMac::GetNumberDevices(const AudioObjectPropertyScope scope,
>+   }
>+ 
>+   if (!listOK) {
>+-    if (deviceIds) {
>+-      free(deviceIds);
>+-      deviceIds = NULL;
>+-    }
>+-
>+     if (bufferList) {
>+       free(bufferList);
>+       bufferList = NULL;
>+@@ -1654,12 +1650,6 @@ int32_t AudioDeviceMac::GetNumberDevices(const AudioObjectPropertyScope scope,
>+     return -1;
>+   }
>+ 
>+-  // Happy ending
>+-  if (deviceIds) {
>+-    free(deviceIds);
>+-    deviceIds = NULL;
>+-  }
>+-
>+   return numberScopedDevices;
>+ }
>+

Actions: View | Formatted Diff | Diff

Attachments on bug 186152: 341679 | 341681 | 341684