After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 300094 - Reload is broken
Reload is broken
Status: RESOLVED FIXED
Product: evince
Classification: Core
Component: general
0.2.x
Other Linux
: High normal
: ---
Assigned To: Evince Maintainers
Evince Maintainers
: 300233 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-04-10 15:43 UTC by Laurent Martelli
Modified: 2005-04-15 23:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to CVS (6.49 KB, patch)
2005-04-11 15:52 UTC, Nickolay V. Shmyrev
none Details | Review
Patch to CVS (6.49 KB, patch)
2005-04-11 15:53 UTC, Nickolay V. Shmyrev
none Details | Review
Updated patch (7.56 KB, patch)
2005-04-11 20:56 UTC, Nickolay V. Shmyrev
committed Details | Review
Test document (24.15 KB, application/pdf)
2005-04-12 11:36 UTC, Laurent Martelli
  Details

Description Laurent Martelli 2005-04-10 15:43:05 UTC
Open a PDF document, press CTRL+R: document is not displayed anymore, you have a
grey window and these errors on stdout:


** (evince:3623): CRITICAL **: PopplerPage*
poppler_document_get_page(PopplerDocument*, int): assertion `0 <= index && index
< poppler_document_get_n_pages (document)' failed

** (evince:3623): CRITICAL **: void
pdf_document_thumbnails_get_dimensions(EvDocumentThumbnails*, int, int, gint*,
gint*): assertion `poppler_page != NULL' failed

** (evince:3623): CRITICAL **: ev_page_cache_get_size: assertion `page >= 0 &&
page < page_cache->n_pages' failed

** (evince:3623): CRITICAL **: ev_pixbuf_cache_set_page_range: assertion
`start_page >= 0 && start_page < ev_page_cache_get_n_pages (page_cache)' failed

** (evince:3623): CRITICAL **: ev_page_cache_get_page_label: assertion `page >=
0 && page < page_cache->n_pages' failed

(evince:3623): Gtk-CRITICAL **: gtk_entry_set_text: assertion `text != NULL' failed

** (evince:3623): CRITICAL **: ev_page_cache_get_page_label: assertion `page >=
0 && page < page_cache->n_pages' failed

(evince:3623): Gtk-CRITICAL **: gtk_entry_set_text: assertion `text != NULL' failed

** (evince:3623): CRITICAL **: ev_page_cache_get_size: assertion `page >= 0 &&
page < page_cache->n_pages' failed

** (evince:3623): CRITICAL **: ev_pixbuf_cache_set_page_range: assertion
`start_page >= 0 && start_page < ev_page_cache_get_n_pages (page_cache)' failed

** (evince:3623): CRITICAL **: ev_page_cache_get_size: assertion `page >= 0 &&
page < page_cache->n_pages' failed

** (evince:3623): CRITICAL **: ev_page_cache_get_size: assertion `page >= 0 &&
page < page_cache->n_pages' failed
Comment 1 Nickolay V. Shmyrev 2005-04-11 15:52:56 UTC
Created attachment 45139 [details] [review]
Patch to CVS

Is your original document has 1 page? That error should be reproduced only with
1-page documents.

Anyhow, this patch should fix the problem.
Comment 2 Nickolay V. Shmyrev 2005-04-11 15:53:48 UTC
Created attachment 45140 [details] [review]
Patch to CVS

Is your original document has 1 page? That error should be reproduced only with
1-page documents.

Anyhow, this patch should fix the problem.
Comment 3 Laurent Martelli 2005-04-11 17:38:34 UTC
Yes, I think it was a one page document. 

I've tested the patch and it works for me. Although I still have a few critical
errors:

** (evince:19249): CRITICAL **: PopplerPage*
poppler_document_get_page(PopplerDocument*, int): assertion `0 <= index && index
< poppler_document_get_n_pages (document)' failed

** (evince:19249): CRITICAL **: void
pdf_document_thumbnails_get_dimensions(EvDocumentThumbnails*, int, int, gint*,
gint*): assertion `poppler_page != NULL' failed

** (evince:19249): CRITICAL **: PopplerPage*
poppler_document_get_page(PopplerDocument*, int): assertion `0 <= index && index
< poppler_document_get_n_pages (document)' failed

** (evince:19249): CRITICAL **: void
pdf_document_thumbnails_get_dimensions(EvDocumentThumbnails*, int, int, gint*,
gint*): assertion `poppler_page != NULL' failed

Comment 4 Nickolay V. Shmyrev 2005-04-11 19:36:37 UTC
Sorry, can you more presizely describe what you are doing to get errors? What is
the sequence of you actions?

Of course, I assume you have patch applied.
Comment 5 Nickolay V. Shmyrev 2005-04-11 20:18:12 UTC
*** Bug 300233 has been marked as a duplicate of this bug. ***
Comment 6 Nickolay V. Shmyrev 2005-04-11 20:56:59 UTC
Created attachment 45149 [details] [review]
Updated patch
Comment 7 Laurent Martelli 2005-04-11 21:13:28 UTC
I should say that I applied your patch against 0.2.0 because I couldn't manage
to compile from CVS.

I also tried your updated patch against 0.20 and I now have these errors when I
open the document (evince <doc.pdf>) or reload it with Ctrl-R:

** (evince:4848): CRITICAL **: PopplerPage*
poppler_document_get_page(PopplerDocument*, int): assertion `0 <= index && index
< poppler_document_get_n_pages (document)' failed

** (evince:4848): CRITICAL **: void
pdf_document_thumbnails_get_dimensions(EvDocumentThumbnails*, int, int, gint*,
gint*): assertion `poppler_page != NULL' failed
Comment 8 Nickolay V. Shmyrev 2005-04-12 10:10:03 UTC
I can't reproduce it neither with 0.2.0 nor with recent CVS. Patch is already
commited. Can you attach problematic document and describe the way you get warnings?
Comment 9 Laurent Martelli 2005-04-12 11:36:35 UTC
Created attachment 45169 [details]
Test document

The errors occur when I open the document using "evince joy_spring_bes.pdf",
with 0.2.0 with the updated patch applied.
Comment 10 Levin Fritz 2005-04-13 19:59:51 UTC
I can confirm this with CVS HEAD: When opening or reloading a 1-page pdf file,
the warnings that Laurent mentioned are printed. Doesn't happen with documents
that have more than one page, though.
Comment 11 Nickolay V. Shmyrev 2005-04-13 20:08:54 UTC
Thanks, this is really 1-page problem. Will fix it soon.
Comment 12 Nickolay V. Shmyrev 2005-04-15 23:39:53 UTC
Should be fixed in CVS