Closed Bug 408256 Opened 17 years ago Closed 17 years ago

Use a constant-size buffer in BMP decoder to reduce fragmentation

Categories

(Core :: Graphics: ImageLib, defect, P1)

defect

Tracking

()

VERIFIED FIXED
mozilla1.9beta3

People

(Reporter: Gavin, Assigned: Gavin)

Details

(Keywords: fixed1.8.0.15, verified1.8.1.12)

Attachments

(1 file)

Attached patch patchSplinter Review
like so!
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attachment #293003 - Flags: superreview?(pavlov)
Attachment #293003 - Flags: review?(dveditz)
Comment on attachment 293003 [details] [diff] [review] patch r/sr=dveditz
Attachment #293003 - Flags: superreview?(pavlov)
Attachment #293003 - Flags: superreview+
Attachment #293003 - Flags: review?(dveditz)
Attachment #293003 - Flags: review+
Flags: blocking1.9+
mozilla/modules/libpr0n/decoders/bmp/nsBMPDecoder.cpp 1.38
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Priority: -- → P1
Resolution: --- → FIXED
Comment on attachment 293003 [details] [diff] [review] patch Would be nice to get this into 2.0.0.12 as well.
Attachment #293003 - Flags: approval1.8.1.12?
Flags: blocking1.8.1.12+
Comment on attachment 293003 [details] [diff] [review] patch approved for 1.8.1.12, a=dveditz for release-drivers
Attachment #293003 - Flags: approval1.8.1.12? → approval1.8.1.12+
mozilla/modules/libpr0n/decoders/bmp/nsBMPDecoder.cpp 1.29.4.1
Keywords: fixed1.8.1.12
Flags: wanted1.8.1.x+
Is this one of these magical fixes where one of you just tells us in QA that it is in or is the verification in bug 408076 also verification of this?
The verification of bug 408076 verifies this bug.
Status: RESOLVED → VERIFIED
Flags: blocking1.8.0.15?
distro patch: blocks 1.8.0.15
Flags: blocking1.8.0.15? → blocking1.8.0.15+
Comment on attachment 293003 [details] [diff] [review] patch a=asac for 1.8.0.15 approving unmodified distro patch.
Attachment #293003 - Flags: approval1.8.0.15+
I landed the patch in bug 408076 on the 1.8.0 branch.
Keywords: fixed1.8.0.15
mozilla/modules/libpr0n/decoders/bmp/nsBMPDecoder.cpp 1.29.12.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: