Title: [198260] branches/safari-601.5.17.2-branch/Source/WebKit/win
Revision
198260
Author
[email protected]
Date
2016-03-15 23:23:37 -0700 (Tue, 15 Mar 2016)

Log Message

Merged r198244.  rdar://problem/25181891

Modified Paths

Diff

Modified: branches/safari-601.5.17.2-branch/Source/WebKit/win/ChangeLog (198259 => 198260)


--- branches/safari-601.5.17.2-branch/Source/WebKit/win/ChangeLog	2016-03-16 06:22:02 UTC (rev 198259)
+++ branches/safari-601.5.17.2-branch/Source/WebKit/win/ChangeLog	2016-03-16 06:23:37 UTC (rev 198260)
@@ -1,3 +1,19 @@
+2016-03-15  Babak Shafiei  <[email protected]>
+
+        Merge r198244.
+
+    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-02-08  Babak Shafiei  <[email protected]>
 
         Merge r187245.

Modified: branches/safari-601.5.17.2-branch/Source/WebKit/win/WebDownloadCFNet.cpp (198259 => 198260)


--- branches/safari-601.5.17.2-branch/Source/WebKit/win/WebDownloadCFNet.cpp	2016-03-16 06:22:02 UTC (rev 198259)
+++ branches/safari-601.5.17.2-branch/Source/WebKit/win/WebDownloadCFNet.cpp	2016-03-16 06:23:37 UTC (rev 198260)
@@ -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