Closed Bug 45895 Opened 25 years ago Closed 25 years ago

Leaking contents of nsPluginInfo struct

Categories

(Core Graveyard :: Plug-ins, defect, P3)

PowerPC
All
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: beard, Assigned: serhunt)

References

()

Details

(Whiteboard: Fix in hand [nsbeta3+])

Attachments

(1 file)

nsPluginFile::FreePluginInfo(nsPluginInfo& info) doesn't free any of the dynamically allocated arrays allocated by nsPluginFile::GetPluginInfo(nsPluginInfo& info). See the BLAME URL above for a pointer to the code that allocates leaking data.
Chris, if you have already filed the same thing for nsPluginDirUNIX.cpp, could you please resolve duplications?
Here's a fix.
Changin platform to all, since UNIX also leaks and Windows need some clean up in nsPluginInfo struct. Fix is in hand so ominating for Beta 3.
Status: NEW → ASSIGNED
Keywords: nsbeta3
OS: Mac System 9.0 → All
Whiteboard: Fix in hand
Target Milestone: --- → M18
Approving for nsbeta3
Whiteboard: Fix in hand → Fix in hand [nsbeta3+]
Checked in. The change also fixes leak on Unix and cleans up nsPluginInfo struct in general. Leak on Windows has already been fixed earlier.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
*** Bug 41942 has been marked as a duplicate of this bug. ***
marking verified based on av's comment.
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: