Closed Bug 1047210 Opened 11 years ago Closed 11 years ago

fix allocation in pkix_pl_CrlDp_Create

Categories

(NSS :: Libraries, defect)

3.16.3
x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
3.16.4

People

(Reporter: sachingpt999, Unassigned, Mentored)

References

Details

Attachments

(1 file)

Attached patch Proposed Patch.Splinter Review
Error: "Port_ArenaZnew" is a function whose sole duty is to assign memory to its first argument equivalent to "sizeof(2nd argument)" Here rdnArena is a pointer to a structure which should be allocated memory equivalent to "sizeof(Certname)", however in current NSS 3.16.3 version it is sizeof(Certname*) which is size of a pointer. NSS 3.16.3 Version code: issuerNameCopy = (CERTName *)PORT_ArenaZNew(rdnArena, CERTName*); Recommended Code: issuerNameCopy = (CERTName *)PORT_ArenaZNew(rdnArena, CERTName);
Attachment #8465958 - Flags: review?(kaie)
Attachment #8465958 - Flags: review?(kaie)
Attachment #8465958 - Flags: review?(kaie)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment on attachment 8465958 [details] [diff] [review] Proposed Patch. r=kaie - thanks for the patch!
Attachment #8465958 - Flags: review?(kaie) → review+
Summary: In file "nss/lib/libpkix/pkix_pl_nss/pki/pkix_pl_crldp.c" function "pkix_pl_CrlDp_Create" on line 125, the amount of memory being allocated to rdnArena is that of a pointer (sizeof(Certname*)) → fix allocation in pkix_pl_CrlDp_Create
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.17
Comment on attachment 8465958 [details] [diff] [review] Proposed Patch. Review of attachment 8465958 [details] [diff] [review]: ----------------------------------------------------------------- r=wtc. I verified there is no other bug of this kind. Kai, please check in this fix for NSS 3.16.4 also.
Attachment #8465958 - Flags: review+
Target Milestone: 3.17 → 3.16.4
Blocks: 1048876
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: