Bug 983766

Summary: nssutil_ReadSecmodDB() leaks memory
Product: Red Hat Enterprise Linux 5 Reporter: Elio Maldonado Batiz <emaldona>
Component: nssAssignee: Elio Maldonado Batiz <emaldona>
Status: CLOSED ERRATA QA Contact: Alicja Kario <hkario>
Severity: medium Docs Contact:
Priority: high    
Version: 5.10CC: amarecek, ebenes, emaldona, eparis, hkario, huzaifas, jkurik, jrieden, kdudka, kengert, ksrot, nobody+bgollahe, ovasik, rrelyea
Target Milestone: rcKeywords: Regression, ZStream
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: nss-3.14.3-13.el5 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 975755 Environment:
Last Closed: 2013-09-30 22:43:46 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 975755, 976572, 987131, 988072, 988083    
Bug Blocks: 927157, 986969    
Attachments:
Description Flags
Switch to using PLArenaPool on blapitest.c none

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