Closed Bug 341115 Opened 18 years ago Closed 18 years ago

Multiple NULL ptr dereferences in nss/lib/base/arena.c

Categories

(NSS :: Libraries, defect, P2)

3.11.1
defect

Tracking

(Not tracked)

RESOLVED FIXED
3.11.3

People

(Reporter: nelson, Assigned: alvolkov.bgs)

Details

(Keywords: coverity, Whiteboard: [CID 290 291 292 293])

Attachments

(1 file)

All these errors are in nss/lib/base/arena.c CID 293 nssArena_Destroy uses a pointer and then checks it for NULL. 523 PR_Lock(arena->lock); 524 if( (PRLock *)NULL == arena->lock ) { CID 291 nssArena_Mark does the same thing 588 PR_Lock(arena->lock); 589 if( (PRLock *)NULL == arena->lock ) { CID 292 nss_ZAlloc does the same thing 911 PR_Lock(arenaOpt->lock); 912 if( (PRLock *)NULL == arenaOpt->lock ) { CID 290 nss_arena_unmark_release does the same thing 671 PR_Lock(arena->lock); 672 if( (PRLock *)NULL == arena->lock ) {
Priority: -- → P2
Whiteboard: [CID 290 291 292 293]
Attached patch fixSplinter Review
Attachment #226524 - Flags: review?(nelson)
Comment on attachment 226524 [details] [diff] [review] fix r=nelson
Attachment #226524 - Flags: review?(nelson) → review+
tip: new revision: 1.9; previous revision: 1.8 3.11 branch: new revision: 1.8.28.1; previous revision: 1.8
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: