nsspkcs5_PKCS12PBE: Variable "A" tracked as NULL was dereferenced.

RESOLVED FIXED in 3.11.1

Status

NSS
Libraries
P2
critical
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: timeless, Assigned: Alexei Volkov)

Tracking

({coverity, crash})

3.11
3.11.1
coverity, crash

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: CID 505, URL)

Attachments

(1 attachment)

937 bytes, patch
Nelson Bolyard (seldom reads bugmail)
: review+
Alexei Volkov
: review+
Details | Diff | Splinter Review
(Reporter)

Description

11 years ago
found by coverity

Updated

11 years ago
Assignee: kengert → nobody
Clearly the code at "loser" should not crash if A is NULL.
Assignee: nobody → alexei.volkov.bugs
OS: Linux → All
Priority: -- → P2
Hardware: PC → All
Target Milestone: --- → 3.11.1
Version: 4.0 → 3.11
(Reporter)

Comment 2

11 years ago
Created attachment 218766 [details] [diff] [review]
null check A ...
Assignee: alexei.volkov.bugs → timeless
Status: NEW → ASSIGNED
Attachment #218766 - Flags: review?
(Reporter)

Updated

11 years ago
Attachment #218766 - Flags: review? → review?(nelson)
(Assignee)

Updated

11 years ago
Attachment #218766 - Flags: review+
Attachment #218766 - Flags: review?(nelson) → review+
(Assignee)

Comment 3

11 years ago
tip:
/cvsroot/mozilla/security/nss/lib/softoken/lowpbe.c,v  <--  lowpbe.c
new revision: 1.16; previous revision: 1.15

3.11 branch:
/cvsroot/mozilla/security/nss/lib/softoken/lowpbe.c,v  <--  lowpbe.c
new revision: 1.14.2.2; previous revision: 1.14.2.1
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
(Assignee)

Updated

11 years ago
Assignee: timeless → alexei.volkov.bugs
CID 505
Whiteboard: CID 505
You need to log in before you can comment on or make changes to this bug.