From 4a63e8efe6eb78688eb7b1229cf85f93677b066b Mon Sep 17 00:00:00 2001
From: Masamichi Hosoda <trueroad@trueroad.jp>
Date: Sat, 20 Aug 2016 19:35:27 +0900
Subject: [PATCH 1/3] Divide Catalog::findDest()

In order to use constructing LinkDest from other functions,
this commit divides Catalog::findDest().
---
 poppler/Catalog.cc | 24 +++++++++++++++++-------
 poppler/Catalog.h  |  2 ++
 2 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/poppler/Catalog.cc b/poppler/Catalog.cc
index a8c96ac..13fbd93 100644
--- a/poppler/Catalog.cc
+++ b/poppler/Catalog.cc
@@ -32,6 +32,7 @@
 // Copyright (C) 2013 José Aliste <jaliste@src.gnome.org>
 // Copyright (C) 2014 Ed Porras <ed@moto-research.com>
 // Copyright (C) 2015 Even Rouault <even.rouault@spatialys.com>
+// Copyright (C) 2016 Masamichi Hosoda <trueroad@trueroad.jp>
 //
 // To see a description of the changes please see the Changelog file that
 // came with your tarball or type make ChangeLog if you are building from git
@@ -451,7 +452,7 @@ int Catalog::findPage(int num, int gen) {
 
 LinkDest *Catalog::findDest(GooString *name) {
   LinkDest *dest;
-  Object obj1, obj2;
+  Object obj1;
   GBool found;
 
   // try named destination dictionary then name tree
@@ -472,12 +473,22 @@ LinkDest *Catalog::findDest(GooString *name) {
   if (!found)
     return NULL;
 
-  // construct LinkDest
+  dest = constructLinkDest(&obj1);
+  obj1.free();
+
+  return dest;
+}
+
+LinkDest *Catalog::constructLinkDest(Object *obj)
+{
+  LinkDest *dest;
+  Object obj2;
+
   dest = NULL;
-  if (obj1.isArray()) {
-    dest = new LinkDest(obj1.getArray());
-  } else if (obj1.isDict()) {
-    if (obj1.dictLookup("D", &obj2)->isArray())
+  if (obj->isArray()) {
+    dest = new LinkDest(obj->getArray());
+  } else if (obj->isDict()) {
+    if (obj->dictLookup("D", &obj2)->isArray())
       dest = new LinkDest(obj2.getArray());
     else
       error(errSyntaxWarning, -1, "Bad named destination value");
@@ -485,7 +496,6 @@ LinkDest *Catalog::findDest(GooString *name) {
   } else {
     error(errSyntaxWarning, -1, "Bad named destination value");
   }
-  obj1.free();
   if (dest && !dest->isOk()) {
     delete dest;
     dest = NULL;
diff --git a/poppler/Catalog.h b/poppler/Catalog.h
index bc9ce20..577416a 100644
--- a/poppler/Catalog.h
+++ b/poppler/Catalog.h
@@ -25,6 +25,7 @@
 // Copyright (C) 2013 Adrian Perez de Castro <aperez@igalia.com>
 // Copyright (C) 2013 Adrian Johnson <ajohnson@redneon.com>
 // Copyright (C) 2013 José Aliste <jaliste@src.gnome.org>
+// Copyright (C) 2016 Masamichi Hosoda <trueroad@trueroad.jp>
 //
 // To see a description of the changes please see the Changelog file that
 // came with your tarball or type make ChangeLog if you are building from git
@@ -263,6 +264,7 @@ private:
   NameTree *getDestNameTree();
   NameTree *getEmbeddedFileNameTree();
   NameTree *getJSNameTree();
+  LinkDest *constructLinkDest(Object *obj);
 #if MULTITHREADED
   GooMutex mutex;
 #endif
-- 
2.8.3