From 4b2e72c4cd7dc9bc17ad78ca983ff884c1e1ee69 Mon Sep 17 00:00:00 2001
From: Jay Berkenbilt <ejb@ql.org>
Date: Thu, 22 Aug 2019 17:53:20 -0400
Subject: Test for direct, rather than resolved nulls in parser

Just because we know an indirect reference is null, doesn't mean we
shouldn't keep it indirect.
---
 libqpdf/QPDFObjectHandle.cc | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

(limited to 'libqpdf/QPDFObjectHandle.cc')

diff --git a/libqpdf/QPDFObjectHandle.cc b/libqpdf/QPDFObjectHandle.cc
index 6240395d..c58675a4 100644
--- a/libqpdf/QPDFObjectHandle.cc
+++ b/libqpdf/QPDFObjectHandle.cc
@@ -263,9 +263,10 @@ QPDFObjectHandle::isBool()
 }
 
 bool
-QPDFObjectHandle::isResolvedNull() const
+QPDFObjectHandle::isDirectNull() const
 {
-    return QPDFObjectTypeAccessor<QPDF_Null>::check(m->obj.getPointer());
+    return (this->m->initialized && (this->m->objid == 0) &&
+            QPDFObjectTypeAccessor<QPDF_Null>::check(m->obj.getPointer()));
 }
 
 bool
-- 
cgit v1.2.3-54-g00ecf