Closed Bug 741218 Opened 12 years ago Closed 12 years ago

Loading a page with content-type application/json causes crash [@ nsHTMLDocument::StartDocumentLoad ]

Categories

(Core :: DOM: HTML Parser, defect)

x86
macOS
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla14

People

(Reporter: simon, Assigned: Ms2ger)

References

Details

(Keywords: crash, testcase)

Crash Data

Attachments

(2 files)

Attached file Test case
When trying to load a resource with content-type application/json directly in the browser (but not via XHR), Nightly crashes. One such crash is

https://crash-stats.mozilla.com/report/index/bp-cc774709-06af-44d5-b653-ddfce2120401

I've attempted to attach a file that will reproduce this to the ticket, but if it doesn't reproduce the issue, I'll post a URL.
Attachment #611288 - Attachment is obsolete: true
That attachment didn't get served as application/json, so try http://www.simonster.com/bug741218.php
Attached patch Patch v1Splinter Review
Assignee: nobody → Ms2ger
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #611291 - Flags: review?(hsivonen)
Comment on attachment 611291 [details] [diff] [review]
Patch v1

Thanks. (My earlier patch for nsHTMLDocument.cpp was probably in flight while JSON landed in nsContentDLF...)
Attachment #611291 - Flags: review?(hsivonen) → review+
Severity: normal → critical
Crash Signature: [@ nsHTMLDocument::StartDocumentLoad]
Keywords: crash, testcase
https://hg.mozilla.org/mozilla-central/rev/4df84fc7cbf1
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
Is this in the most recently released nightly? I want to check to make sure that this crash https://crash-stats.mozilla.com/report/index/bp-e6673058-4911-45cb-b4ba-1a1172120403 is occurring because I don't yet have the patch, and not because there's a different bug floating around.
(In reply to Brandon Savage [:brandon] from comment #6)
> Is this in the most recently released nightly?
The fix will be in Nightly from April 4th.
Attachment #611288 - Attachment is obsolete: false
Attachment #611288 - Attachment mime type: application/octet-stream → application/json
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: