Closed Bug 564374 Opened 11 years ago Closed 10 years ago

sec_pkcs12_decoder_safe_bag_update is forward declared after its implementation

Categories

(NSS :: Libraries, defect, P4)

defect

Tracking

(Not tracked)

RESOLVED FIXED
3.12.7

People

(Reporter: timeless, Assigned: timeless)

References

()

Details

Attachments

(1 file)

Defined as a function in:
    * security/nss/lib/pkcs12/p12d.c
          o line 316 -- sec_pkcs12_decoder_safe_bag_update(void *arg, const char *data, 

Defined as a function prototype in:
    * security/nss/lib/pkcs12/p12d.c
          o line 367 -- sec_pkcs12_decoder_safe_bag_update(void *arg, const char *data,
Taking, since I'm fixing a slew of memory leaks in that same file.
Assignee: nobody → nelson
Attached patch patchSplinter Review
Assignee: nelson → timeless
Status: NEW → ASSIGNED
Attachment #444043 - Flags: review?(nelson)
Comment on attachment 444043 [details] [diff] [review]
patch

Yeah, I'm inclined to move those other forward declarations higher up in the source file, too.
Attachment #444043 - Flags: review?(nelson) → review+
Checking in p12d.c; new revision: 1.48; previous revision: 1.47
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Priority: -- → P4
Target Milestone: --- → 3.12.7
You need to log in before you can comment on or make changes to this bug.