Make conversion between CPDF_Action and its dictionary explicit

core/src/fpdfdoc/doc_action.cpp

 // Copyright 2014 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.
  
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "../../include/fpdfdoc/fpdf_doc.h"
 CPDF_Dest CPDF_Action::GetDest(CPDF_Document* pDoc) const
 {
     if (m_pDict == NULL) {
@@ skipping 76 matching lines @@
             csURI = pURI->GetString("Base") + csURI;
         }
     }
     return csURI;
 }
 FX_DWORD CPDF_ActionFields::GetFieldsCount() const
 {
     if (m_pAction == NULL) {
         return 0;
     }
-    CPDF_Dictionary* pDict = (CPDF_Dictionary*)(*m_pAction);
+    CPDF_Dictionary* pDict = m_pAction->GetDict();
     if (pDict == NULL) {
         return 0;
     }
     CFX_ByteString csType = pDict->GetString("S");
     CPDF_Object* pFields = NULL;
     if (csType == "Hide") {
         pFields = pDict->GetElementValue("T");
     } else {
         pFields = pDict->GetArray("Fields");
     }
     if (pFields == NULL) {
         return 0;
     }
     int iType = pFields->GetType();
     if (iType == PDFOBJ_DICTIONARY) {
         return 1;
     } else if (iType == PDFOBJ_STRING) {
         return 1;
     } else if (iType == PDFOBJ_ARRAY) {
         return ((CPDF_Array*)pFields)->GetCount();
     }
     return 0;
 }
 void CPDF_ActionFields::GetAllFields(CFX_PtrArray& fieldObjects) const
 {
     fieldObjects.RemoveAll();
     if (m_pAction == NULL) {
         return;
     }
-    CPDF_Dictionary* pDict = (CPDF_Dictionary*)(*m_pAction);
+    CPDF_Dictionary* pDict = m_pAction->GetDict();
     if (pDict == NULL) {
         return;
     }
     CFX_ByteString csType = pDict->GetString("S");
     CPDF_Object* pFields = NULL;
     if (csType == "Hide") {
         pFields = pDict->GetElementValue("T");
     } else {
         pFields = pDict->GetArray("Fields");
     }
@@ skipping 12 matching lines @@
                 fieldObjects.Add(pObj);
             }
         }
     }
 }
 CPDF_Object* CPDF_ActionFields::GetField(FX_DWORD iIndex) const
 {
     if (m_pAction == NULL) {
         return NULL;
     }
-    CPDF_Dictionary* pDict = (CPDF_Dictionary*)(*m_pAction);
+    CPDF_Dictionary* pDict = m_pAction->GetDict();
     if (pDict == NULL) {
         return NULL;
     }
     CFX_ByteString csType = pDict->GetString("S");
     CPDF_Object* pFields = NULL;
     if (csType == "Hide") {
         pFields = pDict->GetElementValue("T");
     } else {
         pFields = pDict->GetArray("Fields");
     }
@@ skipping 82 matching lines @@
         return 1;
     }
     if (iObjType == PDFOBJ_ARRAY) {
         return ((CPDF_Array*)pNext)->GetCount();
     }
     return 0;
 }
 CPDF_Action CPDF_Action::GetSubAction(FX_DWORD iIndex) const
 {
     if (m_pDict == NULL || !m_pDict->KeyExist("Next")) {
-        return NULL;
+        return CPDF_Action();
     }
     CPDF_Object* pNext = m_pDict->GetElementValue("Next");
     int iObjType = pNext->GetType();
     if (iObjType == PDFOBJ_DICTIONARY) {
+        CPDF_Dictionary *pDict = static_cast<CPDF_Dictionary*>(pNext);
         if (iIndex == 0) {
-            return (CPDF_Dictionary*)pNext;
+            return CPDF_Action(pDict);
         }
+    } else if (iObjType == PDFOBJ_ARRAY) {
+        CPDF_Array* pArray = static_cast<CPDF_Array*>(pNext);
+        return CPDF_Action(pArray->GetDict(iIndex));
     }
-    if (iObjType == PDFOBJ_ARRAY) {
-        return ((CPDF_Array*)pNext)->GetDict(iIndex);
-    }
-    return NULL;
+    return CPDF_Action();
 }
 const FX_CHAR* g_sAATypes[] = {"E", "X", "D", "U", "Fo", "Bl", "PO", "PC", "PV", "PI",
                                "O", "C",
                                "K", "F", "V", "C",
                                "WC", "WS", "DS", "WP", "DP",
                                ""
                               };
 FX_BOOL CPDF_AAction::ActionExist(AActionType eType) const
 {
     if (m_pDict == NULL) {
         return FALSE;
     }
     return m_pDict->KeyExist(g_sAATypes[(int)eType]);
 }
 CPDF_Action CPDF_AAction::GetAction(AActionType eType) const
 {
-    if (m_pDict == NULL) {
-        return NULL;
+    if (!m_pDict) {
+        return CPDF_Action();
     }
-    return m_pDict->GetDict(g_sAATypes[(int)eType]);
+    return CPDF_Action(m_pDict->GetDict(g_sAATypes[(int)eType]));
 }
 FX_POSITION CPDF_AAction::GetStartPos() const
 {
     if (m_pDict == NULL) {
         return NULL;
     }
     return m_pDict->GetStartPos();
 }
 CPDF_Action CPDF_AAction::GetNextAction(FX_POSITION& pos, AActionType& eType) const
 {
     if (m_pDict == NULL) {
-        return NULL;
+        return CPDF_Action();
     }
     CFX_ByteString csKey;
     CPDF_Object* pObj = m_pDict->GetNextElement(pos, csKey);
-    if (pObj != NULL) {
-        CPDF_Object* pDirect = pObj->GetDirect();
-        if (pDirect != NULL && pDirect->GetType() == PDFOBJ_DICTIONARY) {
-            int i = 0;
-            while (g_sAATypes[i][0] != '\0') {
-                if (csKey == g_sAATypes[i]) {
-                    break;
-                }
-                i ++;
-            }
-            eType = (AActionType)i;
-            return (CPDF_Dictionary*)pDirect;
+    if (!pObj) {
+        return CPDF_Action();
+    }
+    CPDF_Object* pDirect = pObj->GetDirect();
+    if (!pDirect || pDirect->GetType() != PDFOBJ_DICTIONARY) {
+        return CPDF_Action();
+    }
+    int i = 0;
+    while (g_sAATypes[i][0] != '\0') {
+        if (csKey == g_sAATypes[i]) {
+            break;
         }
+        i++;
     }
-    return NULL;
+    eType = (AActionType)i;
+    return CPDF_Action(static_cast<CPDF_Dictionary*>(pDirect));
 }
 CPDF_DocJSActions::CPDF_DocJSActions(CPDF_Document* pDoc)
 {
     m_pDocument = pDoc;
 }
 int CPDF_DocJSActions::CountJSActions() const
 {
     ASSERT(m_pDocument != NULL);
     CPDF_NameTree name_tree(m_pDocument, FX_BSTRC("JavaScript"));
     return name_tree.GetCount();
 }
 CPDF_Action CPDF_DocJSActions::GetJSAction(int index, CFX_ByteString& csName) const
 {
     ASSERT(m_pDocument != NULL);
     CPDF_NameTree name_tree(m_pDocument, FX_BSTRC("JavaScript"));
     CPDF_Object *pAction = name_tree.LookupValue(index, csName);
     if (pAction == NULL || pAction->GetType() != PDFOBJ_DICTIONARY) {
-        return NULL;
+        return CPDF_Action();
     }
-    return pAction->GetDict();
+    return CPDF_Action(pAction->GetDict());
 }
 CPDF_Action CPDF_DocJSActions::GetJSAction(const CFX_ByteString& csName) const
 {
     ASSERT(m_pDocument != NULL);
     CPDF_NameTree name_tree(m_pDocument, FX_BSTRC("JavaScript"));
     CPDF_Object *pAction = name_tree.LookupValue(csName);
     if (pAction == NULL || pAction->GetType() != PDFOBJ_DICTIONARY) {
-        return NULL;
+        return CPDF_Action();
     }
-    return pAction->GetDict();
+    return CPDF_Action(pAction->GetDict());
 }
 int CPDF_DocJSActions::FindJSAction(const CFX_ByteString& csName) const
 {
     ASSERT(m_pDocument != NULL);
     CPDF_NameTree name_tree(m_pDocument, FX_BSTRC("JavaScript"));
     return name_tree.GetIndex(csName);
 }