Closed Bug 336483 Opened 15 years ago Closed 15 years ago

Coverity crash or hopefully deadcode [@ crmf_copy_popoprivkey] because Pointer "destPrivKey" dereferenced before NULL check

Categories

(NSS :: Libraries, defect, P2)

Product:
NSS
Component:
Libraries
Version:
3.11
Platform:
All
Linux
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
3.11.2

People

(Reporter: timeless, Assigned: alvolkov.bgs)

References

(
URL
)

Details

(Keywords: coverity, crash, Whiteboard: CID 630)

Crash Data

this looks like something that could be asserted, but at the very least, the null check in rv != SECSuccess is pointless.
Hardware: PC → All
Target Milestone: --- → 3.11.2
Priority: -- → P2
Assignee: nobody → alexei.volkov.bugs
fixed as a part of the fix for bug 336482
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Oops, reopened wrong bug.
Status: REOPENED → RESOLVED
Closed: 15 years ago → 15 years ago
Resolution: --- → FIXED
CID 630
Whiteboard: CID 630
Summary: crash or hopefully deadcode [@ crmf_copy_popoprivkey] because Pointer "destPrivKey" dereferenced before NULL check → Coverity crash or hopefully deadcode [@ crmf_copy_popoprivkey] because Pointer "destPrivKey" dereferenced before NULL check
Crash Signature: [@ crmf_copy_popoprivkey]
You need to log in before you can comment on or make changes to this bug.