Fabio D'Urso <[email protected]> writes:

> The attached patch fixes a crash in Page::removeAnnot, which blindly assumes 
> /Annots only contains Ref objects. Some PDF creators embed links (which are 
> Dictionaries) in the /Annots array, and this makes us abort() with
>
> Error (0): Call to Object where the object was type 7, not the expected type 9
>
> when we stumble on such a link.
>
> This patch adds a check to skip non-Ref entries (regular annotations *are* 
> Ref 
> entries).

Looks good, pushed to git master.

> Test document (recycled from another bug, turns out it wasn't that useless :D)
>  https://bugs.freedesktop.org/attachment.cgi?id=63393&action=edit
>
> Steps to reproduce:
>  1. Open the document
>  2. Add new annotation
>  3. Remove it
>  4. abort() gets called
>
> Thanks,
> Fabio

Thanks!
-- 
Carlos Garcia Campos
PGP key: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x523E6462

Attachment: pgplluKrYBJsR.pgp
Description: PGP signature

_______________________________________________
poppler mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/poppler

Reply via email to