Check the validity of pointers before they are utilized

fpdfsdk/src/fpdfppo.cpp

Index: fpdfsdk/src/fpdfppo.cpp
diff --git a/fpdfsdk/src/fpdfppo.cpp b/fpdfsdk/src/fpdfppo.cpp
index b8d2125b7dab667e0e0d9d50aa708474309757b6..a98640758150b4cb1590bfc8caac3d4562b4d036 100644
--- a/fpdfsdk/src/fpdfppo.cpp
+++ b/fpdfsdk/src/fpdfppo.cpp
@@ -313,9 +313,14 @@ int	CPDF_PageOrganizer::GetNewObjId(CPDF_Document *pDoc, CFX_MapPtrToPtr* pMapPt
 	}
 	else
 	{
-		CPDF_Object* pClone  = pRef->GetDirect()->Clone();
-		if(!pClone)			
+		CPDF_Object* pClone  = NULL;

[Tom Sepez, 2014/09/08 17:36:20]

nit: maybe this reads clearer.  Your call.

CPDF_Object* pDirect = pRef->GetDirect();
if (!pDirect) {
  return 0;
}
CPDF_Object* pClone = pDirect->Clone();
if (!pClone) {
  return 0;
}

+		CPDF_Object* pDirect = pRef->GetDirect();
+		if (pDirect) {
+			pClone = pDirect->Clone();
+		}
+		if(!pClone) {
 			return 0;
+		}
 		
 		if(pClone->GetType() == PDFOBJ_DICTIONARY)
 		{