Leak in nsCrypto::GenerateCRMFRequest (security/manager/ssl/src/nsCrypto.cpp)

RESOLVED FIXED

Status

Core Graveyard
Security: UI
--
minor
RESOLVED FIXED
12 years ago
a year ago

People

(Reporter: Kenneth Herron, Assigned: kaie)

Tracking

({coverity, fixed1.8.1})

Trunk
coverity, fixed1.8.1

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug], URL)

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
This is coverity issue 576. Please refer to the sample URL. At lines 1584-1586, the |keyids| array allocated at line 1552 is leaked.
(Reporter)

Updated

12 years ago
Whiteboard: [good first bug]
(Assignee)

Comment 1

12 years ago
Created attachment 221503 [details] [diff] [review]
Patch v1
Attachment #221503 - Flags: review?(rrelyea)

Comment 2

12 years ago
Comment on attachment 221503 [details] [diff] [review]
Patch v1

r+ = rrelyea
Attachment #221503 - Flags: review?(rrelyea) → review+
(Assignee)

Comment 3

12 years ago
fixed on trunk
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
(Assignee)

Comment 4

12 years ago
Comment on attachment 221503 [details] [diff] [review]
Patch v1

should this leak fix land on the 1.8 branch?
Attachment #221503 - Flags: approval1.8.1?

Comment 5

12 years ago
Comment on attachment 221503 [details] [diff] [review]
Patch v1

a=darin on behalf of drivers
Attachment #221503 - Flags: approval1.8.1? → approval1.8.1+
(Assignee)

Comment 6

12 years ago
fixed1.8.1
Keywords: fixed1.8.1
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.