NSArray leaks seen in Safari, allocated under WKIconDatabaseTryCopyCGImageArrayForURL
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 13 Dec 2016 20:23:07 +0000 (20:23 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 13 Dec 2016 20:23:07 +0000 (20:23 +0000)
https://bugs.webkit.org/show_bug.cgi?id=165809

Patch by Joseph Pecoraro <pecoraro@apple.com> on 2016-12-13
Reviewed by Dan Bernstein.

* UIProcess/API/C/cg/WKIconDatabaseCG.cpp:
(WKIconDatabaseTryCopyCGImageArrayForURL):
Avoid an extra retain on already newly created array.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@209769 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/API/C/cg/WKIconDatabaseCG.cpp

index 666a337..fea57af 100644 (file)
@@ -1,3 +1,14 @@
+2016-12-13  Joseph Pecoraro  <pecoraro@apple.com>
+
+        NSArray leaks seen in Safari, allocated under WKIconDatabaseTryCopyCGImageArrayForURL
+        https://bugs.webkit.org/show_bug.cgi?id=165809
+
+        Reviewed by Dan Bernstein.
+
+        * UIProcess/API/C/cg/WKIconDatabaseCG.cpp:
+        (WKIconDatabaseTryCopyCGImageArrayForURL):
+        Avoid an extra retain on already newly created array.
+
 2016-12-13  Eric Carlson  <eric.carlson@apple.com>
 
         Annotate MediaStream and WebRTC idl with EnabledAtRuntime flag
index 3056336..a279af6 100644 (file)
@@ -54,6 +54,5 @@ CFArrayRef WKIconDatabaseTryCopyCGImageArrayForURL(WKIconDatabaseRef iconDatabas
     for (auto nativeImage : nativeImages)
         CFArrayAppendValue(array, nativeImage.get());
     
-    return static_cast<CFArrayRef>(CFRetain(array));
+    return static_cast<CFArrayRef>(array);
 }
-