76f8c5
From 1e6541a3a443db7afd3046483c0fd994b31d128c Mon Sep 17 00:00:00 2001
76f8c5
From: Jason Crain <jason@aquaticape.us>
76f8c5
Date: Sat, 18 Apr 2015 12:44:47 -0500
76f8c5
Subject: [PATCH] glib: Fix segfault when creating PopplerAction
76f8c5
76f8c5
Screen annotations and form fields currently pass a NULL pointer to
76f8c5
_poppler_action_new.  Pass the PopplerDocument instead.
76f8c5
76f8c5
Bug #90093
76f8c5
---
76f8c5
 glib/poppler-annot.cc      | 4 ++--
76f8c5
 glib/poppler-form-field.cc | 2 +-
76f8c5
 glib/poppler-page.cc       | 2 +-
76f8c5
 glib/poppler-private.h     | 2 +-
76f8c5
 4 files changed, 5 insertions(+), 5 deletions(-)
76f8c5
76f8c5
diff --git a/glib/poppler-annot.cc b/glib/poppler-annot.cc
76f8c5
index 15d48c5..312aa31 100644
76f8c5
--- a/glib/poppler-annot.cc
76f8c5
+++ b/glib/poppler-annot.cc
76f8c5
@@ -559,7 +559,7 @@ poppler_annot_screen_class_init (PopplerAnnotScreenClass *klass)
76f8c5
 }
76f8c5
 
76f8c5
 PopplerAnnot *
76f8c5
-_poppler_annot_screen_new (Annot *annot)
76f8c5
+_poppler_annot_screen_new (PopplerDocument *doc, Annot *annot)
76f8c5
 {
76f8c5
   PopplerAnnot *poppler_annot;
76f8c5
   AnnotScreen  *annot_screen;
76f8c5
@@ -569,7 +569,7 @@ _poppler_annot_screen_new (Annot *annot)
76f8c5
   annot_screen = static_cast<AnnotScreen *>(poppler_annot->annot);
76f8c5
   action = annot_screen->getAction();
76f8c5
   if (action)
76f8c5
-    POPPLER_ANNOT_SCREEN (poppler_annot)->action = _poppler_action_new (NULL, action, NULL);
76f8c5
+    POPPLER_ANNOT_SCREEN (poppler_annot)->action = _poppler_action_new (doc, action, NULL);
76f8c5
 
76f8c5
   return poppler_annot;
76f8c5
 }
76f8c5
diff --git a/glib/poppler-form-field.cc b/glib/poppler-form-field.cc
76f8c5
index 5687799..df184dd 100644
76f8c5
--- a/glib/poppler-form-field.cc
76f8c5
+++ b/glib/poppler-form-field.cc
76f8c5
@@ -188,7 +188,7 @@ poppler_form_field_get_action (PopplerFormField *field)
76f8c5
   if (!action)
76f8c5
     return NULL;
76f8c5
 
76f8c5
-  field->action = _poppler_action_new (NULL, action, NULL);
76f8c5
+  field->action = _poppler_action_new (field->document, action, NULL);
76f8c5
 
76f8c5
   return field->action;
76f8c5
 }
76f8c5
diff --git a/glib/poppler-page.cc b/glib/poppler-page.cc
76f8c5
index db2387a..1025393 100644
76f8c5
--- a/glib/poppler-page.cc
76f8c5
+++ b/glib/poppler-page.cc
76f8c5
@@ -1399,7 +1399,7 @@ poppler_page_get_annot_mapping (PopplerPage *page)
76f8c5
         mapping->annot = _poppler_annot_movie_new (annot);
76f8c5
 	break;
76f8c5
       case Annot::typeScreen:
76f8c5
-        mapping->annot = _poppler_annot_screen_new (annot);
76f8c5
+        mapping->annot = _poppler_annot_screen_new (page->document, annot);
76f8c5
 	break;
76f8c5
       case Annot::typeLine:
76f8c5
         mapping->annot = _poppler_annot_line_new (annot);
76f8c5
diff --git a/glib/poppler-private.h b/glib/poppler-private.h
76f8c5
index 874cfdb..9abdd7c 100644
76f8c5
--- a/glib/poppler-private.h
76f8c5
+++ b/glib/poppler-private.h
76f8c5
@@ -130,7 +130,7 @@ PopplerAnnot      *_poppler_annot_free_text_new (Annot *annot);
76f8c5
 PopplerAnnot      *_poppler_annot_text_markup_new     (Annot *annot);
76f8c5
 PopplerAnnot      *_poppler_annot_file_attachment_new (Annot *annot);
76f8c5
 PopplerAnnot      *_poppler_annot_movie_new (Annot *annot);
76f8c5
-PopplerAnnot      *_poppler_annot_screen_new (Annot *annot);
76f8c5
+PopplerAnnot      *_poppler_annot_screen_new (PopplerDocument *doc, Annot *annot);
76f8c5
 PopplerAnnot      *_poppler_annot_line_new (Annot *annot);
76f8c5
 PopplerAnnot      *_poppler_annot_circle_new (Annot *annot);
76f8c5
 PopplerAnnot      *_poppler_annot_square_new (Annot *annot);
76f8c5
-- 
76f8c5
2.1.4
76f8c5