Closed Bug 1097283 Opened 5 years ago Closed 5 years ago

Review NS_ABORT_OOM callsites for count/bytes mismatches

Categories

(Core :: XPCOM, defect)

36 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla36

People

(Reporter: dmajor, Assigned: dmajor)

Details

Attachments

(1 file)

I just made this mistake myself by copying nearby code in bug 1096155. As atonement, I'm going to fix all of them!
Contains a drive-by fix in nsTextFragment.h: Presumably the append operation uses both lengths.

Open to other names for AllocFailed.
Attachment #8521070 - Flags: review?(nfroyd)
Comment on attachment 8521070 [details] [diff] [review]
Pound of cure + ounce of prevention

Nice.
Attachment #8521070 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/2cedc0acd40b
Assignee: nobody → dmajor
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.