Closed Bug 4358 Opened 26 years ago Closed 25 years ago

UMR: nsHTMLButtonElement::HandleDOMElement()

Categories

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

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: bruce, Assigned: pollmann)

References

()

Details

It seems that 'aEventStatus' can have an uninitialized value still when coming out of the call to mInner.HandleDOMEvent() in nsHTMLButtonElement::HandleDOMEvent(). This happens when submitting via the HTML button. The stack trace is: nsHTMLButtonElement::HandleDOMEvent() nsHTMLButtonControlFrame::MouseClicked() nsHTMLButtonControlFrame::HandleEvent() PresShell::HandleEvent() Build from March 28, 1999. Solaris 2.6, gcc 2.7.2.3, Purify.
Assignee: karnaze → evaughan
Eric V, if you don't want this, please reassign it to Eric Pollmann.
Target Milestone: M4
What does the 'UMR:' mean in the summary? uninitialized memory reference? (just guessing). If this is a crash, the severity should be major or critical. targetting m4.
Assignee: evaughan → pollmann
I have not worked with any of the html4 content model code. I have not idea.
Target Milestone: M4 → M5
Marking M5
Status: NEW → ASSIGNED
Target Milestone: M5 → M7
Target Milestone: M7 → M8
I'm arbitrarily declaring M8 my UMR/MLK milestone. :)
Target Milestone: M8 → M10
I still haven't got Purify working on Solaris, and won't receive Purify for NT for a few weeks. Marking these M10
UMR = Uninitialized Memory Read. This looks like a simple lack of initialization in nsGenericElement::HandleDOMEvent().
Working on a fix...
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Kevin fixed this on 9/13. nsButtonControlFrame::MouseClicked now contains this line: nsEventStatus status = nsEventStatus_eIgnore; Before any of the calls to HandleDOMEvent. This is a code level change and is not verifiable by running the program (except for less random incorrect event behaviour.) :)
If it isn't verifiable by running the app... I'm going to just go on ahead and mark it as verified. bruce, if you see any further problems in this area, please reopen. thanks!
Status: RESOLVED → VERIFIED
Component: HTML: Form Submission → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.