RESOLVED FIXED Bug 132106
[Cocoa] fix CF leaks found by code inspection 
https://bugs.webkit.org/show_bug.cgi?id=132106
Summary [Cocoa] fix CF leaks found by code inspection
Darin Adler
Reported 2014-04-23 21:11:31 PDT
[Cocoa] fix CF leaks found by code inspection
Attachments
Patch (18.82 KB, patch)
2014-04-23 21:14 PDT, Darin Adler 		kling: review+
kling: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2014-04-23 21:14:46 PDT
Created attachment 230044 [details] Patch
Andreas Kling
Comment 2 2014-04-23 21:27:27 PDT
Comment on attachment 230044 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=230044&action=review r=me. Nice finds! > Source/WebCore/plugins/mac/PluginPackageMac.cpp:138 > + RetainPtr<CFStringRef> homeDir = adoptCF(homeDirectoryPath().createCFString()); We don't need adoptCF() here; createCFString() returns a RetainPtr<CFStringRef>. > Source/WebCore/plugins/mac/PluginPackageMac.cpp:255 > + RetainPtr<CFStringRef> path = adoptCF(m_path.createCFString()); Ditto.
Darin Adler
Comment 3 2014-04-23 21:46:52 PDT
Committed r167746: <http://trac.webkit.org/changeset/167746>
Darin Adler
Comment 4 2014-04-23 21:47:25 PDT
Comment on attachment 230044 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=230044&action=review >> Source/WebCore/plugins/mac/PluginPackageMac.cpp:138 >> + RetainPtr<CFStringRef> homeDir = adoptCF(homeDirectoryPath().createCFString()); > > We don't need adoptCF() here; createCFString() returns a RetainPtr<CFStringRef>. Not even sure how this compiles.
Darin Adler
Comment 5 2014-04-23 21:48:42 PDT
*** Bug 132108 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.