393183 - Optstate not freed in ocspclnt.

Status: RESOLVED FIXED

(NSS :: Tools, defect, P3)

Description

[Slavomir Katuscak]

Memory Leak (mel):
Found leaked block of size 36 bytes at address 0x5e0600
At time of allocation, the call stack was:
    [1] PR_Malloc() at line 467 in "prmem.c"
    [2] PL_CreateLongOptState() at line 99 in "plgetopt.c"
    [3] PL_CreateOptState() at line 76 in "plgetopt.c"
    [4] main() at line 1027 in "ocspclnt.c"

Memory Leak (mel):
Found leaked block of size 20 bytes at address 0x5e05d0
At time of allocation, the call stack was:
    [1] calloc() at 0xe0954ab4
    [2] PR_Calloc() at line 475 in "prmem.c"
    [3] PL_CreateLongOptState() at line 92 in "plgetopt.c"
    [4] PL_CreateOptState() at line 76 in "plgetopt.c"
    [5] main() at line 1027 in "ocspclnt.c"

Allocated variable:
   1027     optstate = PL_CreateOptState (argc, argv, "AHLPR:S:V:d:l:pr:s:t:u:w:");

Comment 1

[Nelson Bolyard (seldom reads bugmail)]

While you're at it, this change to ocspclnt.c will help, too:

     if (NSS_Shutdown () != SECSuccess) {
-       exit(1);
+       reval = 1;
     }

Comment 2

[Slavomir Katuscak]

Attachment: Patch.

Comment 3

[Slavomir Katuscak]

Trunk:
Checking in ocspclnt.c;
/cvsroot/mozilla/security/nss/cmd/ocspclnt/ocspclnt.c,v  <--  ocspclnt.c
new revision: 1.12; previous revision: 1.11
done

Branch:
Checking in ocspclnt.c;
/cvsroot/mozilla/security/nss/cmd/ocspclnt/ocspclnt.c,v  <--  ocspclnt.c
new revision: 1.7.28.2; previous revision: 1.7.28.1
done