Closed
Bug 4358
Opened 26 years ago
Closed 25 years ago
UMR: nsHTMLButtonElement::HandleDOMElement()
Categories
(Core :: DOM: Core & HTML, defect, P3)
Core
DOM: Core & HTML
Tracking
()
VERIFIED
FIXED
M10
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.
Updated•26 years ago
|
Assignee: karnaze → evaughan
Comment 1•26 years ago
|
||
Eric V, if you don't want this, please reassign it to Eric Pollmann.
Updated•26 years ago
|
Target Milestone: M4
Comment 2•26 years ago
|
||
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.
Updated•26 years ago
|
Assignee: evaughan → pollmann
Comment 3•26 years ago
|
||
I have not worked with any of the html4 content model code. I have not idea.
Assignee | ||
Updated•26 years ago
|
Target Milestone: M4 → M5
Assignee | ||
Comment 4•26 years ago
|
||
Marking M5
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M5 → M7
Assignee | ||
Updated•26 years ago
|
Target Milestone: M7 → M8
Assignee | ||
Comment 5•26 years ago
|
||
I'm arbitrarily declaring M8 my UMR/MLK milestone. :)
Assignee | ||
Updated•26 years ago
|
Target Milestone: M8 → M10
Assignee | ||
Comment 6•26 years ago
|
||
I still haven't got Purify working on Solaris, and won't receive Purify for NT
for a few weeks. Marking these M10
Assignee | ||
Comment 7•25 years ago
|
||
UMR = Uninitialized Memory Read. This looks like a simple lack of
initialization in nsGenericElement::HandleDOMEvent().
Assignee | ||
Comment 8•25 years ago
|
||
Working on a fix...
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•25 years ago
|
||
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.) :)
Comment 10•25 years ago
|
||
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!
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
Component: HTML: Form Submission → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•