Closed Bug 17953 Opened 26 years ago Closed 26 years ago

[mlk] nsBrowserAppCore::UpdateGoMenu() leaks nsXULElements

Categories

(SeaMonkey :: General, defect, P1)

Product:
SeaMonkey
Component:
General
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: waterson, Assigned: waterson)

References

(
URL
)

Details

Attachments

(1 file)

proposed patch
2.21 KB, patch
Details | Diff | Splinter Review
use of nsCOMPtr fixes this. attaching patch for review: law/radha, could you take a look? thanks, chris
Attached patch proposed patch — — Splinter Review
FWIW, this accounts for 3 of the 25 nsXULElements leaked.
Priority: P3 → P1
Target Milestone: M12
Your proposed changes look good to me.
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
QA Contact: leger → waterson
waterson, can you mark this Verified now? looks good?
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: