Commit 4bf178bd authored by relyea%netscape.com's avatar relyea%netscape.com
Browse files 

fix Bug 93989 Memory leak when adding a module.
parent b3b596f6

security/nss/lib/pk11wrap/pk11util.c

+3 −1
Original line number Diff line number Diff line
@@ -329,6 +329,7 @@ SECMOD_DeleteInternalModule(char *name) { 
SECStatus

SECMOD_AddModule(SECMODModule *newModule) {

    SECStatus rv;

    SECMODModule *oldModule;

    int i;



    /* Test if a module w/ the same name already exists */
@@ -336,7 +337,8 @@ SECMOD_AddModule(SECMODModule *newModule) { 
    /* We should probably add a new return value such as */

    /* SECDublicateModule, but to minimize ripples, I'll */

    /* give SECWouldBlock a new meaning */

    if (SECMOD_FindModule(newModule->commonName)) {

    if ((oldModule = SECMOD_FindModule(newModule->commonName)) != NULL) {

	SECMOD_DestroyModule(oldModule);

        return SECWouldBlock;

        /* module already exists. */

    }