Closed Bug 52387 Opened 26 years ago Closed 26 years ago

FMM: Freeing mismatched memory in ~nsHTMLContentSinkStream

Categories

(Core :: DOM: HTML Parser, defect, P3)

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: jband_mozilla, Assigned: rickg)

Details

Attachments

(2 files)

nsHTMLContentSinkStream::mBuffer uses new[] and delete[] everywhere except in the class dtor where nsMemory::Free is erroneously used. I'll attach a Purify trace and proposed fix. Give me a r= and I'll check it in.
Attached patch proposed fix — — Splinter Review
Thanks JBand. I'll update the file.
Status: NEW → ASSIGNED
I've tried your patch and it's fine. I'll land it with my next update (today if the tree opens).
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
updated qa contact.
QA Contact: janc → bsharma
Marking verified as per above developer comments.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: