Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Atril crash on pdf (poppler) #222

Closed
emanuele-f opened this issue Sep 30, 2016 · 9 comments
Closed

Atril crash on pdf (poppler) #222

emanuele-f opened this issue Sep 30, 2016 · 9 comments

Comments

@emanuele-f
Copy link

emanuele-f commented Sep 30, 2016

Atril crashes with the following PDF atril_core_dump.pdf.

As reported in #137, crash occurs in poppler.

System is latest 64 bit archlinux
Poppler version is 0.47.0-1

@monsta
Copy link
Contributor

monsta commented Oct 9, 2016

Poppler 0.48.0 has been released yesterday. I suggest trying that document with it.

@monsta
Copy link
Contributor

monsta commented Oct 23, 2016

So, does it still happen with 0.48.0?

@emanuele-f
Copy link
Author

Yes, it still happens

@monsta
Copy link
Contributor

monsta commented Oct 23, 2016

I just found out there's a patch for poppler to fix this, but looks like it wasn't accepted upstream (yet):
https://bugs.freedesktop.org/show_bug.cgi?id=97870#c3

I only was able to reproduce the crash with atril-thumbnailer (while loading the doc in Atril itself doesn't crash it for me). The patch fixes the crash for good.

@emanuele-f
Copy link
Author

I've verified the following:

  • atril version 1.16.0 (from arch repo) with poppler 0.48.0 crashes
  • atril version 1.14.2 with poppler 0.47.0-1 does not crash (VM)
  • atril master branch from github with poppler 0.47.0-1 does not crash (VM)
  • atril master branch from github with poppler 0.48.0 does not crash (VM)

poppler version I indicate is the one installed in the system, although I cannot see libpoppler as a dependency with ldd command. Is poppler statically linked to atril?

@monsta
Copy link
Contributor

monsta commented Oct 23, 2016

Indeed, ldd doesn't show poppler to me either. But the dynamic library is the one that's used, that's for sure. Otherwise we wouldn't see libpoppler.so.NN in the stacktrace.

It seems that the crash is best reproducible with atril-thumbnailer:

$ atril-thumbnailer atril_core_dump.pdf some_output_file

@emanuele-f
Copy link
Author

I've just tested with linked patch https://bugs.freedesktop.org/attachment.cgi?id=126698 and I report same experience as yours: atril does not crash, while atri-thumbnailer crashes. The function where it crashes it's still cmsGetColorSpace as before the patch.

@monsta
Copy link
Contributor

monsta commented Jan 25, 2017

The patch is finally included upstream (avalable in poppler 0.50), plus a correction for it:

I still can't reproduce the crash of Atril itself, and the patch (with correction) still fixes crash of atril-thumbnailer.

@monsta
Copy link
Contributor

monsta commented Feb 14, 2017

Anyway, the issue is (was) in poppler, so please ask upstream developers if there's still some problem with it. Upstream bug report is at https://bugs.freedesktop.org/show_bug.cgi?id=97870.

@monsta monsta closed this as completed Feb 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants