Bug 40008

Summary: Leaks in listDirectory
Product: WebKit Reporter: Alejandro G. Castro <alex>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Proposed patch xan.lopez: review+

Description Alejandro G. Castro 2010-06-01 10:21:52 PDT
valgrind show a couple of leaks in the file FileSystemGtk.cpp:

==18726==    at 0x4C221A7: malloc (vg_replace_malloc.c:195)
==18726==    by 0xFF6E816: g_malloc (gmem.c:131)
==18726==    by 0xFF78082: g_pattern_spec_new (gpattern.c:268)
==18726==    by 0x5955C32: WebCore::listDirectory(WebCore::String const&, WebCore::String const&) (FileSystemGtk.cpp:211)

==18726==    at 0x4C221A7: malloc (vg_replace_malloc.c:195)
==18726==    by 0xFF6E816: g_malloc (gmem.c:131)
==18726==    by 0xFFA3B8F: g_uri_unescape_segment (gurifuncs.c:98)
==18726==    by 0xFFA3C88: g_uri_unescape_string (gurifuncs.c:159)
==18726==    by 0x59556A5: WebCore::filenameFromString(WebCore::String const&) (FileSystemGtk.cpp:60)
==18726==    by 0x5955BEB: WebCore::listDirectory(WebCore::String const&, WebCore::String const&) (FileSystemGtk.cpp:206)
Comment 1 Alejandro G. Castro 2010-06-01 10:25:11 PDT
Created attachment 57563 [details]
Proposed patch
Comment 2 Xan Lopez 2010-06-01 10:27:13 PDT
Comment on attachment 57563 [details]
Proposed patch

ok
Comment 3 Alejandro G. Castro 2010-06-04 03:57:18 PDT
Landed r60673