Title: [198244] trunk/Source/WebKit/win
Revision
198244
Author
[email protected]
Date
2016-03-15 17:00:26 -0700 (Tue, 15 Mar 2016)

Log Message

[Win] Correct double-release of CFURLConnectionRef
https://bugs.webkit.org/show_bug.cgi?id=155515
<rdar://problem/25159143>

Reviewed by Tim Horton.

Tested by http/tests/download suite.

* WebDownloadCFNet.cpp: Remove extra CFRelease.

Modified Paths

Diff

Modified: trunk/Source/WebKit/win/ChangeLog (198243 => 198244)


--- trunk/Source/WebKit/win/ChangeLog	2016-03-15 23:59:24 UTC (rev 198243)
+++ trunk/Source/WebKit/win/ChangeLog	2016-03-16 00:00:26 UTC (rev 198244)
@@ -1,3 +1,15 @@
+2016-03-15  Brent Fulgham  <[email protected]>
+
+        [Win] Correct double-release of CFURLConnectionRef
+        https://bugs.webkit.org/show_bug.cgi?id=155515
+        <rdar://problem/25159143>
+
+        Reviewed by Tim Horton.
+
+        Tested by http/tests/download suite.
+
+        * WebDownloadCFNet.cpp: Remove extra CFRelease.
+
 2016-03-14  Per Arne Vollan  <[email protected]>
 
         [WinCairo] Compile fix.

Modified: trunk/Source/WebKit/win/WebDownloadCFNet.cpp (198243 => 198244)


--- trunk/Source/WebKit/win/WebDownloadCFNet.cpp	2016-03-15 23:59:24 UTC (rev 198243)
+++ trunk/Source/WebKit/win/WebDownloadCFNet.cpp	2016-03-16 00:00:26 UTC (rev 198244)
@@ -96,7 +96,6 @@
     // or it fails to creating and we have a now-useless connection with a dangling ref. 
     // Either way, we need to release the connection to balance out ref counts
     handle->releaseConnectionForDownload();
-    CFRelease(connection);
 }
 
 void WebDownload::init(const URL& url, IWebDownloadDelegate* delegate)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to