Closed Bug 137249 Opened 23 years ago Closed 23 years ago

listbox widget with invalid children crashes

Categories

(Core :: XUL, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

()

VERIFIED DUPLICATE of bug 125689

People

(Reporter: markh, Assigned: jag+mozilla)

Details

(Keywords: crash)

Attachments

(2 files)

If you create a listbox with invalid content, Mozilla crashes. Eg: <listbox> <checkbox label="foo"/> </listbox> will crash. I'm not sure exactly what I expect that to do, but I don't expect it to crash :) Attaching complete crashing XUL.
Keywords: crash, patch
Attached patch Proposed patchSplinter Review
Patch that fixes the NULL pointer de-reference. There is no mBox object with invalid children.
Seems like a reasonable patch. hewitt, jag, varga: wanna review this patch?
Keywords: review
Comment on attachment 79055 [details] [diff] [review] Proposed patch looks good r=varga
Attachment #79055 - Flags: review+
*** This bug has been marked as a duplicate of 125689 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
okay
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: