Bug 717913

Summary: [PEM] various flaws detected by Coverity
Product: Red Hat Enterprise Linux 6 Reporter: Michal Luscon <mluscon>
Component: nssAssignee: Elio Maldonado Batiz <emaldona>
Status: CLOSED ERRATA QA Contact: Aleš Mareček <amarecek>
Severity: medium Docs Contact:
Priority: high    
Version: 6.1CC: amarecek, emaldona, jrieden, kdudka, praiskup, rrelyea
Target Milestone: rcKeywords: Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: nss-3.13.3-5.el6 Doc Type: Bug Fix
Doc Text:
No documentation needed
Story Points: ---
Clone Of:
: 806051 (view as bug list) Environment:
Last Closed: 2012-06-20 07:23:12 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:    
Bug Blocks: 806051    
Attachments:
Description Flags
proposed fix
emaldona: review+
Added defects between releases from rhel-6.2/rhel-6.3 none

Description Michal Luscon 2011-06-30 12:09:03 UTC
Description of problem:

/mozilla/security/nss/lib/ckfw/pem/util.c:209: - Dynamically allocated variable der is not freed above goto loser statements (lines #217, #225, ...).

Version-Release number of selected component (if applicable):
3.12.9-9

Additional info:
This defect was probably introduced by Red Hat patches.

Comment 6 Kamil Dudka 2011-08-31 11:09:35 UTC
Created attachment 520793 [details]
proposed fix

Comment 7 RHEL Program Management 2011-10-07 16:01:19 UTC
Since RHEL 6.2 External Beta has begun, and this bug remains
unresolved, it has been rejected as it is not proposed as
exception or blocker.

Red Hat invites you to ask your support representative to
propose this request, if appropriate and relevant, in the
next release of Red Hat Enterprise Linux.

Comment 8 Kamil Dudka 2011-12-08 20:53:50 UTC
*** Bug 748011 has been marked as a duplicate of this bug. ***

Comment 11 Elio Maldonado Batiz 2012-03-16 01:19:27 UTC
Comment on attachment 520793 [details]
proposed fix

r+ from me. Need to rename if to follow new conventions.

Comment 13 Pavel Raiskup 2012-04-03 11:09:54 UTC
Created attachment 574826 [details]
Added defects between releases from rhel-6.2/rhel-6.3

I checked new scan results of nss-3.13.3-5.el6 and there is no PEM related
defect inside.  I seems this could be switched to VERIFIED.

But there have occurred another potential bugs between the version nss-3.12.10
from RHEL-6.2 and nss-3.13.3 in RHEL-6.3.  List of added defects is attached.
From this list it seems to be real the following bug:

    * nss-3.13.3/mozilla/security/nss/lib/ssl/ssl3con.c:8123:
        unitialized variable 'desc' when the goto target 'alert_loser' is
        followed from the line 7905.

Other added defects against el6.2 seem to be low-priority warnings for
programmers typos.  Because mentioned defect is not related to this BZ# it may
be considered to open new bug for it of course.

Comment 16 Elio Maldonado Batiz 2012-04-26 17:50:51 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
No documentation needed

Comment 20 errata-xmlrpc 2012-06-20 07:23:12 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/RHSA-2012-0973.html