Closed
Bug 393181
Opened 17 years ago
Closed 16 years ago
Memory leaks in ocspclnt in calls to CERT_SetOCSPDefaultResponder
Categories
(NSS :: Libraries, defect, P2)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 403888
3.11.10
People
(Reporter: slavomir.katuscak+mozilla, Assigned: julien.pierre)
Details
(Keywords: memory-leak)
Found some memory leaks in CERT_SetOCSPDefaultResponder: Memory Leak (mel): Found leaked block of size 38 bytes at address 0x5f7bc0 At time of allocation, the call stack was: [1] PR_Malloc() at line 467 in "prmem.c" [2] PORT_Alloc() at line 113 in "secport.c" [3] PORT_Strdup() at line 171 in "secport.c" [4] CERT_SetOCSPDefaultResponder() at line 5063 in "ocsp.c" [5] main() at line 1212 in "ocspclnt.c" Memory Leak (mel): Found leaked block of size 25 bytes at address 0x6058f0 At time of allocation, the call stack was: [1] PR_Malloc() at line 467 in "prmem.c" [2] PORT_Alloc() at line 113 in "secport.c" [3] PORT_Strdup() at line 171 in "secport.c" [4] CERT_SetOCSPDefaultResponder() at line 5064 in "ocsp.c" [5] main() at line 1212 in "ocspclnt.c" Memory Leak (mel): Found leaked block of size 16 bytes at address 0x5fbd88 At time of allocation, the call stack was: [1] calloc() at 0xe0954ab4 [2] PR_Calloc() at line 475 in "prmem.c" [3] PORT_ZAlloc() at line 140 in "secport.c" [4] ocsp_InitStatusChecking() at line 4939 in "ocsp.c" [5] CERT_SetOCSPDefaultResponder() at line 5076 in "ocsp.c" [6] main() at line 1212 in "ocspclnt.c" Memory Leak (mel): Found leaked block of size 12 bytes at address 0x5fbe78 At time of allocation, the call stack was: [1] calloc() at 0xe0954ab4 [2] PR_Calloc() at line 475 in "prmem.c" [3] PORT_ZAlloc() at line 140 in "secport.c" [4] ocsp_InitStatusChecking() at line 4935 in "ocsp.c" [5] CERT_SetOCSPDefaultResponder() at line 5076 in "ocsp.c" [6] main() at line 1212 in "ocspclnt.c" What is allocated and not freed: CERT_SetOCSPDefaultResponder 5063 url_copy = PORT_Strdup(url); 5064 name_copy = PORT_Strdup(name); ocsp_InitStatusChecking (called directly from CERT_SetOCSPDefaultResponder): 4935 statusConfig = PORT_ZNew(CERTStatusConfig); 4939 statusContext = PORT_ZNew(ocspCheckingContext);
Comment 1•17 years ago
|
||
Slavo, we need to know what programs called CERT_SetOCSPDefaultResponder() That information is not in the stacks above.
Keywords: mlk
Summary: Memory leaks in CERT_SetOCSPDefaultResponder. → Memory leaks in programs that call CERT_SetOCSPDefaultResponder.
Reporter | ||
Comment 2•17 years ago
|
||
It's from ocspclnt. You can see it also in every stack on the last line - file which contains main(). [5] main() at line 1212 in "ocspclnt.c" --> ocspclnt
Comment 3•17 years ago
|
||
If this is merely a leak in the ocsp test program, then it is only P3, but if it is a leak in the library, then it is P2 for 3.12. For now, I'm marking it assuming that it is a library leak.
Priority: -- → P2
Summary: Memory leaks in programs that call CERT_SetOCSPDefaultResponder. → Memory leaks in ocspclnt in calls to CERT_SetOCSPDefaultResponder
Target Milestone: --- → 3.12
Reporter | ||
Comment 5•17 years ago
|
||
After adding OCSP tests to branch bug found also there.
Target Milestone: 3.12 → 3.11.9
Assignee | ||
Updated•16 years ago
|
Target Milestone: 3.11.9 → 3.11.10
Assignee | ||
Comment 6•16 years ago
|
||
I think is a dupe of 403888 , although the leak was triggered by CERT_EnableOCSPChecking in that bug. The fix was checked in on the trunk. I checked the trunk tinderbox, and there is no longer a leak in CERT_SetOCSPDefaultResponder , whereas the branch tinderbox still reports it in the ignored stacks list. So, I will mark this as a dupe of 403888 and retarget it to 3.11.10 .
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → DUPLICATE
Reporter | ||
Comment 7•16 years ago
|
||
As this is duplicate of 403888 which is already fixed, I'm removing this bug from list of ignored leaks.
You need to log in
before you can comment on or make changes to this bug.
Description
•