Closed
Bug 45432
Opened 24 years ago
Closed 24 years ago
Leaking PRLibrary if a library won't load
Categories
(NSPR :: NSPR, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
4.1
People
(Reporter: beard, Assigned: wtc)
References
()
Details
Attachments
(1 file)
|
810 bytes,
patch
|
Details | Diff | Splinter Review |
See the blame URL above. If the library fails to load on line 929, the PRLibrary object is leaked. This is rare, but I'm seeing it in Mozilla builds when libraries fail to load during auto registration.
|
Assignee | |
Comment 1•24 years ago
|
||
I'm going to attach a patch that fixes this memory leak and a similar leak. Please review and test it. (I'm not set up to build on Mac right now.) Thanks.
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•24 years ago
|
||
|
|
Assignee | |
Updated•24 years ago
|
Target Milestone: --- → 4.1
|
|
Assignee | |
Comment 4•24 years ago
|
||
I checked in the fix on the main trunk. /cvsroot/mozilla/nsprpub/pr/src/linking/prlink.c, revision 3.43
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
|
Reporter | |
Comment 5•24 years ago
|
||
This isn't checked into the branch that Mozilla uses, so the leak is still present in Mozilla.
Status: RESOLVED → REOPENED
OS: All
Resolution: FIXED → ---
|
|
Assignee | |
Comment 6•24 years ago
|
||
I merged the fix on the NSPRPUB_CLIENT_BRANCH. /cvsroot/mozilla/nsprpub/pr/src/linking/prlink.c, revision 3.34.2.8.
Status: REOPENED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•