Closed Bug 337086 Opened 15 years ago Closed 15 years ago

Coverity 446, dead code in mozilla/security/nss/lib/certhigh/ocsp.c

Categories

(NSS :: Libraries, defect, P3)

3.11
defect

Tracking

(Not tracked)

RESOLVED FIXED
3.11.2

People

(Reporter: jonsmirl, Assigned: alvolkov.bgs)

Details

(Keywords: coverity, Whiteboard: [CID 446 843])

Attachments

(1 file)

it is not possible for statusContext to be null in the error case

@@ -3764,8 +3764,6 @@
     return SECSuccess;

 loser:
-    if (statusContext != NULL)
-       PORT_Free(statusContext);
     if (statusConfig != NULL)
        PORT_Free(statusConfig);
     return SECFailure;
Severity: normal → trivial
Priority: -- → P3
Attachment #221265 - Flags: review?(wtchang)
Attachment #221265 - Flags: review?(wtchang) → review+
trunk
/cvsroot/mozilla/security/nss/lib/certhigh/ocsp.c,v  <--  ocsp.c
new revision: 1.25; previous revision: 1.24

3.11 branch
/cvsroot/mozilla/security/nss/lib/certhigh/ocsp.c,v  <--  ocsp.c
new revision: 1.21.2.6; previous revision: 1.21.2.5
Assignee: nobody → alexei.volkov.bugs
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Whiteboard: [CID 446]
Also CID 843
Whiteboard: [CID 446] → [CID 446 843]
You need to log in before you can comment on or make changes to this bug.