CHECK for re-addition of indirect objects

core/fpdfapi/parser/cpdf_document_unittest.cpp

 // Copyright 2016 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/fpdfapi/parser/cpdf_document.h"
 
 #include <memory>
 
 #include "core/fpdfapi/cpdf_modulemgr.h"
 #include "core/fpdfapi/parser/cpdf_array.h"
@@ skipping 51 matching lines @@
     justSix->AddReference(this, AddIndirectObject(CreateNumberedPage(6)));
     CPDF_Dictionary* branch4 = CreatePageTreeNode(justSix, this, 1);
 
     CPDF_Array* allPages = new CPDF_Array();
     allPages->AddReference(this, branch2->GetObjNum());
     allPages->AddReference(this, branch3->GetObjNum());
     allPages->AddReference(this, branch4->GetObjNum());
     CPDF_Dictionary* pagesDict = CreatePageTreeNode(allPages, this, 7);
 
     m_pOwnedRootDict.reset(new CPDF_Dictionary());
-    m_pOwnedRootDict->SetReferenceFor("Pages", this,
-                                      AddIndirectObject(pagesDict));
+    m_pOwnedRootDict->SetReferenceFor("Pages", this, pagesDict->GetObjNum());
     m_pRootDict = m_pOwnedRootDict.get();
     m_PageList.SetSize(7);
   }
 
  private:
   std::unique_ptr<CPDF_Dictionary> m_pOwnedRootDict;
 };
 }  // namespace
 
 class cpdf_document_test : public testing::Test {
@@ skipping 46 matching lines @@
   const int test_page_num = 33;
 
   EXPECT_FALSE(document.IsPageLoaded(test_page_num));
   EXPECT_EQ(nullptr, document.GetPage(test_page_num));
 
   document.SetPageObjNum(test_page_num, obj_num);
 
   EXPECT_TRUE(document.IsPageLoaded(test_page_num));
   EXPECT_EQ(page_stub, document.GetPage(test_page_num));
 }