Closed Bug 283533 Opened 20 years ago Closed 20 years ago

OOM crash [@ XPCIDispatchClassInfo::GetInterfaces]

Categories

(Core :: XPConnect, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: timeless, Assigned: bastiaan)

References

()

Details

(Keywords: crash)

Crash Data

Attachments

(1 file, 1 obsolete file)

note that cleanup for this case needs to properly release the partially constructed array.
Ugh, I forgot to check the return from clone
Status: NEW → ASSIGNED
Attached patch fix (obsolete) — Splinter Review
Attachment #187845 - Flags: superreview?(shaver)
Attachment #187845 - Flags: review?(shaver)
Should we set *count to zero on failure? Or since we're returning an error all bets are off on out parameters? I'll repeat the usual style nit about removing spaces in things like "if (" which.
Attached patch address commentsSplinter Review
Assignee: dbradley → b.jacques
Attachment #187845 - Attachment is obsolete: true
Attachment #187851 - Flags: superreview?(shaver)
Attachment #187851 - Flags: review?(dbradley)
Attachment #187845 - Flags: superreview?(shaver)
Attachment #187845 - Flags: review?(shaver)
Comment on attachment 187851 [details] [diff] [review] address comments r+sr=shaver
Attachment #187851 - Flags: superreview?(shaver)
Attachment #187851 - Flags: superreview+
Attachment #187851 - Flags: review?(dbradley)
Attachment #187851 - Flags: review+
Attachment #187851 - Flags: approval1.8b3?
Comment on attachment 187851 [details] [diff] [review] address comments a=bsmedberg, please land expeditiously
Attachment #187851 - Flags: approval1.8b3? → approval1.8b3+
Be good to include dbradley, nominal module owner. /be
Checked in by timeless (2005-07-06 10:47).
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Crash Signature: [@ XPCIDispatchClassInfo::GetInterfaces]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: