Bug 983766 - nssutil_ReadSecmodDB() leaks memory
Summary: nssutil_ReadSecmodDB() leaks memory
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: nss
Version: 5.10
Hardware: All
OS: Linux
high
medium
Target Milestone: rc
: ---
Assignee: Elio Maldonado Batiz
QA Contact: Alicja Kario
URL:
Whiteboard:
Depends On: 975755 976572 987131 988072 988083
Blocks: 927157 986969
TreeView+ depends on / blocked
 
Reported: 2013-07-11 22:47 UTC by Elio Maldonado Batiz
Modified: 2013-09-30 22:43 UTC (History)
14 users (show)

Fixed In Version: nss-3.14.3-13.el5
Doc Type: Bug Fix
Doc Text:
Clone Of: 975755
Environment:
Last Closed: 2013-09-30 22:43:46 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Switch to using PLArenaPool on blapitest.c (9.78 KB, patch)
2013-07-19 20:18 UTC, Elio Maldonado Batiz
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2013:1318 0 normal SHIPPED_LIVE nss bug fix and enhancement update 2013-09-30 21:13:17 UTC

Comment 8 Elio Maldonado Batiz 2013-07-19 20:18:19 UTC
Created attachment 775949 [details]
Switch to using PLArenaPool on blapitest.c

This patch gets rid of sporadic test core dumps that have been has been observed on AES GCM tests in 64-bit platforms.

Comment 9 Elio Maldonado Batiz 2013-07-19 21:01:29 UTC
This  patch is a subset of the last patch from the upstream bug https://bugzilla.mozilla.org/show_bug.cgi?id=802430
https://bug802430.bugzilla.mozilla.org/attachment.cgi?id=744975
See various comments made on Bug 976572.

Comment 10 Elio Maldonado Batiz 2013-07-20 12:59:10 UTC
This patch hasn't solved the problem, nor has a more complete one I tried. Worth noting is the fact that we get the core dumps even though all the tests pass.

Comment 12 Karel Srot 2013-07-22 07:16:16 UTC
Elio,
may I ask for a status update? Are you going to update the patch? I am confused by your comment #c10 while switching the bug to the MODIFIED state.

Comment 16 Elio Maldonado Batiz 2013-07-22 14:49:03 UTC
(In reply to Karel Srot from comment #12)
> Elio,
> may I ask for a status update? Are you going to update the patch? I am
> confused by your comment #c10 while switching the bug to the MODIFIED state.

I have expanded the patch and it still fails some times. Now that Bob is back from PTO we will be working on this problem together. Yes, it can be confusing and I should have been clearer. I believe that the problem of the sporadic core dumps is orthogonal to the memory leaks we are trying to solve here. I marked it as modified because it was modified. The memory leak patch stands in its own merits and I did not want to get in the way in case you wanted to do some verification of the patch. I understand if you prefer to wait until we have come to a solution of the other problem and have done the respin.

Comment 20 errata-xmlrpc 2013-09-30 22:43:46 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2013-1318.html


Note You need to log in before you can comment on or make changes to this bug.