Closed
Bug 93288
Opened 23 years ago
Closed 23 years ago
SECMOD_AddNewModule leaks on failure
Categories
(NSS :: Libraries, defect)
Tracking
(Not tracked)
People
(Reporter: dbaron, Assigned: wtc)
Details
(Keywords: memory-leak)
secmod_FindExternalRoot calls SECMOD_AddNewModule a number of times expecting it
to fail most of the time. However, SECMOD_AddNewModule leaks on failure. This
results in a bunch of module arenas being leaked on startup in Mozilla.
The following patch fixes the leak for me.
Index: pk11util.c
===================================================================
RCS file: /cvsroot/mozilla/security/nss/lib/pk11wrap/pk11util.c,v
retrieving revision 1.7
diff -u -d -r1.7 pk11util.c
--- pk11util.c 2001/02/06 00:11:24 1.7
+++ pk11util.c 2001/08/02 18:42:00
@@ -506,6 +506,7 @@
if (rv != SECSuccess) {
/* SECFailure: failed to add module, corrupt or missing module
etc. */
/* SECBlock: a module with the same name already exists */
+ SECMOD_DestroyModule(module);
return rv;
} else { /* successfully added module */
/* turn on SSL cipher enable flags */
Assignee | ||
Comment 1•23 years ago
|
||
David, this bug is already fixed. The cvs tag on your
mozilla/security/nss/lib/pk11wrap/pk11util.c should be
NSS_CLIENT_TAG, revision 1.7.2.1.
*** This bug has been marked as a duplicate of 78467 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Version: unspecified → 3.2.1
Reporter | ||
Comment 2•23 years ago
|
||
Hmmm. That's what I get for using MOZ_NSS_AUTOCONF. I'm on the NSS_CLIENT_BRANCH.
Assignee | ||
Comment 3•23 years ago
|
||
I see. Don't set MOZ_NSS_AUTOCONF. It is an experimental
build option and is not supported by the NSS team.
You need to log in
before you can comment on or make changes to this bug.
Description
•