Closed
Bug 2234
Opened 26 years ago
Closed 26 years ago
Calling a form submission through JS crashes the viewer
Categories
(Core :: DOM: Core & HTML, defect, P1)
Tracking
()
VERIFIED
FIXED
People
(Reporter: ernie, Assigned: pollmann)
References
()
Details
1/05,1/07 builds on NT
When I submit a form through a JS function call (using DOM),
it crashes the viewer. If I submit the form manually via
a submit button, no crash occurs.
Here are some code snippets:
<script>
function submitResults()
{
document.forms.item(0).submit();
}
</script>
<form name="results" action="/ernie/cgi-bin/writeresults.cgi" method="post">
...
<input type=button onClick=submitResults() value=submitResults()>
<input type=submit>
</form>
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•26 years ago
|
||
Well, we're dereferencing null there alright... Seems that we're explicitly
passing in nsnull to a function that then does a QI on it:
In nsHTMLFormElement::Submit() we do:
result = formMan->OnSubmit(context, nsnull);
http://cvs-mirror.mozilla.org/webtools/lxr/source/layout/html/content/src/nsHTMLFormElement.cpp#291
Then in nsFormFrame::OnSubmit(aPresContext, aFrame) we do:
aFrame->QueryInterface(kIFormControlFrameIID, (void**)&fcFrame);
http://cvs-mirror.mozilla.org/webtools/lxr/source/layout/html/forms/src/nsFormFrame.cpp#468
I passed in the current frame being submitted and that stopped the crash, but I
don't know if that's "the right thing" so I'm holding back until I learn more...
Assignee | ||
Comment 3•26 years ago
|
||
Checked in a fix.
Assignee | ||
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Updated•26 years ago
|
QA Contact: 4015
Comment 4•26 years ago
|
||
assigning Gerardo as QA Assigned
Comment 5•26 years ago
|
||
using 2/03/99 build, loading the page, selecting each of the buttons does not
crash on win95. Need to verify on linux and mac.
Gerardo -- please verify on linux
Greg -- please verify on mac
thanks
Feb 3 Mac - initial button pressing not working properly, 1st button does not
create text in the field, typing causes text to appear overlapping the Back
Forward buttons, the second and third buttons, submitresults and submitquery
contact the server but then nothing happens. When I try to quit by clicking
the close box, the app hangs. Trying to Macsbug out the first time resulted in
crashing my entire machine. Me thinks we still have a problem here.
Status: RESOLVED → REOPENED
Assignee | ||
Updated•26 years ago
|
Resolution: FIXED → ---
Assignee | ||
Comment 7•26 years ago
|
||
The CGI on snorkel encounters a server error when running, which may be
partially to blame for this crash. (It's returning an error code to viewer)
I set up a functioning test case on blueviper:
http://blueviper/forms/submit.html
This worked for me on Windows and Linux. Does it work on mac?
If the Mac browser is crashing on a server HTTP error code, that may a bug that
in netlib. ???
Status: REOPENED → RESOLVED
Closed: 26 years ago → 26 years ago
Resolution: --- → FIXED
Ok, the crash is caused by something else. In Feb 4 Seamonkey with new test case
this appears to be ok. The rendering of text in the wrong place is a layout
problem separate from this bug as well. Marking this as fixed for Mac.
Verified Linux as well with Feb 4. More layout problems than Mac and same hang
on close box but submission is working.
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
•