Detect/Open incomplete documents

Bug #1382839 reported by Razi Alavizadeh
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
qpdfview
Fix Released
Medium
Adam Reichold

Bug Description

I have a DjVu document that is corrupted, currently when opening it qpdfview crashes.

Indeed crash is when "DocumentView::preparePages()" is in progress, because:
m_pages.count() is 547 but m_pages.at(index) for index > 141 is null.

I think
- At least: qpdfview should cancel loading document and notify user that the document is corrupted and not a bug of qpdfview.

- And it's better if: it loads document and shows suitable replacement for pages that are null.

Revision history for this message
Adam Reichold (adamreichold) wrote :

Hello Razi,

trunk revision 1739 should take a more transactional approach and only load documents if all the indicated pages can actually be loaded. Please test that this handles the case you describe.

I am also firmly against trying to fix up or partially display broken documents as this invites all sorts of corner cases and I find not loading them in the first a much cleaner solution.

Best regards, Adam,

Changed in qpdfview:
status: New → Fix Committed
importance: Undecided → Medium
assignee: nobody → Adam Reichold (adamreichold)
milestone: none → 0.4.13
Revision history for this message
Razi Alavizadeh (srazi) wrote :

Hello Adam,

Works great, thanks.

Best Regards,
Razi.

Changed in qpdfview:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.