Closed Bug 336483 Opened 18 years ago Closed 18 years ago

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

Categories

(NSS :: Libraries, defect, P2)

3.11
All
Linux
defect

Tracking

(Not tracked)

RESOLVED FIXED
3.11.2

People

(Reporter: timeless, Assigned: alvolkov.bgs)

References

()

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: 18 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Oops, reopened wrong bug.
Status: REOPENED → RESOLVED
Closed: 18 years ago18 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.