Closed Bug 1716047 Opened 3 years ago Closed 3 years ago

The browser crashes when loading a specific PDF + XFA file

Categories

(Firefox :: PDF Viewer, defect, P1)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
Tracking Status
firefox-esr78 --- disabled
firefox89 --- disabled
firefox90 --- unaffected
firefox91 --- verified

People

(Reporter: danibodea, Assigned: calixte)

References

Details

Attachments

(2 files)

Note

  • When the user activates XFA support and attempts to open a specific PDF with XFA file, he will notice that a very long loading is observed, but eventually crashes.

Affected versions

  • Nightly v91.0a1 2021-06-10

Affected platforms

  • all

Steps to reproduce

  1. Launch browser
  2. Flip pdfjs.enableXFA to true in about:config
  3. Attempt to open the "REQUEST FOR A CHANGE OF SEX OR GENDER IDENTIFIER" from www.canada.ca

Expected result

  • The PDF is properly displayed.

Actual result

  • The browser loads for a long time and eventually crashes.

Regression range

  • Unfortunately, mozregression kept getting stuck and I could not finish the process, so I got all the info in the log and put it here:
  • Nightly v90.0a1 (2021-05-03) displays the "unsupported" message.
  • Nightly v90.0a1 (2021-05-20) correctly displays the PDF file.
  • Nightly v91.0a1 (2021-06-01, 06-09) correctly displays the PDF file.
  • Nightly v91.0a1 (2021-06-10) shows the issue described above.
  • Regression range:
    2021-06-11T20:58:51.882000: INFO : Narrowed nightly regression window from [2021-06-08, 2021-06-10] (2 days) to [2021-06-09, 2021-06-10] (1 days) (~0 steps left)
    Bisecting on mozilla-central [578715b6 - 8508c35e]

(In reply to Bodea Daniel [:danibodea] from comment #0)

  • Nightly v90.0a1 (2021-05-03) displays the "unsupported" message.
  • Nightly v90.0a1 (2021-05-20) correctly displays the PDF file.

This is unexpected, it should still be showing the "unsupported" message :S

Keywords: access
Severity: S2 → S1
Priority: -- → P1
Assignee: nobody → cdenizet
Status: NEW → ASSIGNED

Can you link the PR which fixed this?

Flags: needinfo?(cdenizet)
Depends on: 1717132

This should now be fixed, Ralu can you verify?

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Flags: needinfo?(raluca.popovici)
Resolution: --- → FIXED

Ah sorry, this was filed by Daniel.

Flags: needinfo?(raluca.popovici) → needinfo?(daniel.bodea)

I can confirm that this issue has been fixed with the latest PR. Thank you!

Status: RESOLVED → VERIFIED
Flags: needinfo?(daniel.bodea)
Flags: needinfo?(cdenizet)
QA Whiteboard: [pdf_xfa_generic]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: