UMR: nsHTMLButtonElement::HandleDOMElement()

VERIFIED FIXED in M10

Status

()

P3
normal
VERIFIED FIXED
20 years ago
20 years ago

People

(Reporter: bruce, Assigned: pollmann)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

20 years ago
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

20 years ago
Assignee: karnaze → evaughan

Comment 1

20 years ago
Eric V, if you don't want this, please reassign it to Eric Pollmann.

Updated

20 years ago
Target Milestone: M4

Comment 2

20 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

20 years ago
Assignee: evaughan → pollmann

Comment 3

20 years ago
I have not worked with any of the html4 content model code. I have not idea.
(Assignee)

Updated

20 years ago
Target Milestone: M4 → M5
(Assignee)

Comment 4

20 years ago
Marking M5
(Assignee)

Updated

20 years ago
Status: NEW → ASSIGNED
Target Milestone: M5 → M7
(Assignee)

Updated

20 years ago
Target Milestone: M7 → M8
(Assignee)

Comment 5

20 years ago
I'm arbitrarily declaring M8 my UMR/MLK milestone.  :)
(Assignee)

Updated

20 years ago
Target Milestone: M8 → M10
(Assignee)

Comment 6

20 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

20 years ago
UMR = Uninitialized Memory Read.  This looks like a simple lack of
initialization in nsGenericElement::HandleDOMEvent().
(Assignee)

Comment 8

20 years ago
Working on a fix...
(Assignee)

Updated

20 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 20 years ago
Resolution: --- → FIXED
(Assignee)

Comment 9

20 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

20 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

20 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.