Memory leak mozilla/widget/src/windows/nsFileWidget.cpp function nsFileWidget::GetFolder calls the Win32 API function ::SHBrowseForFolder which returns a pointer of type LPITEMIDLIST. This is at line 326. I believe the caller is responsible for freeing the memory pointed to. This is not happening.
reassigning to sdagley. Steve, if this is non-trivial please let me know.
Assignee: trudelle → sdagley
http://support.microsoft.com/support/kb/articles/Q179/3/78.ASP shows use of SHGetMalloc to get the shell’s IMalloc interface and use of this interface to free the result returned from SHBrowseForFolder.
*IGNORE* - massive spam changing open XPToolkit bug's QA contact to firstname.lastname@example.org
QA Contact: paulmac → jrgm
reassigning to danm
Assignee: sdagley → danm
Mass-moving all M16 non-feature bugs to M17, which we still consider to be part of beta2
Target Milestone: M16 → M17
moving to m18, since it is reported as minor
Whiteboard: How big a leak is this?
Target Milestone: M17 → M18
mass-moving all bugs to m21 that are not dofood+, or nsbeta2+
Target Milestone: M18 → M21
Whiteboard: How big a leak is this? → [nsbeta3-][need info]How big a leak is this?
Target Milestone: M21 → M19
Assignee: danm → valeski
I'll apply the patch.
Whiteboard: [nsbeta3-][need info]How big a leak is this? → [nsbeta3+][need info]How big a leak is this?
fix is in.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
verified that patch is in with version 3.37 in the tree. Marking VERIFIED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.