20797 - Crashes when viewing CNN Headline News Mail

Status: VERIFIED FIXED

(Core :: DOM: HTML Parser, defect, P2)

Description

[Alex]

Crashes when trying to load my CNN Headline News Mail I regularily recieve in
my email. This crashes only when viewing through webmail, I have not been able
to test viewing it with Messenger. Test case attached below.

Crashes consistently on my Windoze 98 PC, 96 MB RAM, running Build 1999120315
and I believe it crashes on the past previous builds also.

As noted, I will attach testcase.

Comment 1

[Alex]

Attachment: Testcase for the descripted crash

Comment 2

[Mats Palmgren (inactive)]

Attachment: Minimal testcase, distilled from prev. attached page

Comment 3

[Mats Palmgren (inactive)]

Reassigning to Parser, this is causing the crash (see Minimal testcase):
<CENTER>
  <table><tr><td>
</CENTER>
  </td></tr></table>

Comment 4

[Mats Palmgren (inactive)]

My testcase isn't crashing like I said. It's Windows 98 that makes *anything*
crash when it feels like it. Sorry all.

Comment 5

[Mats Palmgren (inactive)]

Attachment: Minimal testcase, second try...

Comment 6

[Mats Palmgren (inactive)]

The new testcase shows the bug:
<TABLE>
  <TR><TD>
    <CENTER>
      <table border=1>
        <tr><td>NASA probe silent after Mars landing</td></tr>
        <!-- Missing tr,td start tags here -->
          Take a virtual peek into the future
        </td></tr>
      </table>
    </CENTER>
  </TD></TR>
</TABLE>

I think it is Parser after all, maybe a duplicate of bug 20645?

Comment 7

[rickg]

This is crashing in the content sink, and happens when bad content gets moved
out of a table.

Comment 8

[leger]

Updating QA Contact.

Comment 9

[Jan Carpenter]

Talkback Incident ID 1931547

Comment 10

[rickg]

Moving to m13.

Comment 11

[rickg]

Crashes in savedtokens, again.

Comment 12

[rickg]

*** Bug 21679 has been marked as a duplicate of this bug. ***

Comment 13

[rickg]

*** Bug 20645 has been marked as a duplicate of this bug. ***

Comment 14

[harishd]

*** Bug 23007 has been marked as a duplicate of this bug. ***

Comment 15

[Crysgem]

The MICROSOFT WINDOWS 98 2000010508 build crashes at
http://www.reston.com/nasa/watch.html, presumably due to the same demon.

Comment 16

[harishd]

Great test case.  Helped me in fixing the bug fast.

Marking FIXED.

Comment 17

[harishd]

*** Bug 21845 has been marked as a duplicate of this bug. ***

Comment 18

[Jan Carpenter]

Verified fixt.

Comment 19

[Pulsebot]

Pushed by ffxbld-merge:
https://hg.mozilla.org/releases/mozilla-beta/rev/6e973cfae3d4
[fenix] Close https://github.com/mozilla-mobile/fenix/issues/20797: Fix intermittent test failures in ShortcutsSuggestionProviderTest