Fix stack overflow in object Clone() functions

core/fpdfapi/fpdf_parser/cpdf_number.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.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "core/fpdfapi/fpdf_parser/include/cpdf_number.h"
 
 CPDF_Number::CPDF_Number() : m_bInteger(true), m_Integer(0) {}
 
 CPDF_Number::CPDF_Number(int value) : m_bInteger(true), m_Integer(value) {}
 
 CPDF_Number::CPDF_Number(FX_FLOAT value) : m_bInteger(false), m_Float(value) {}
 
 CPDF_Number::CPDF_Number(const CFX_ByteStringC& str)
     : m_bInteger(FX_atonum(str, &m_Integer)) {}
 
 CPDF_Number::~CPDF_Number() {}
 
 CPDF_Object::Type CPDF_Number::GetType() const {
   return NUMBER;
 }
 
-CPDF_Object* CPDF_Number::Clone(FX_BOOL bDirect) const {
+CPDF_Object* CPDF_Number::Clone() const {
   return m_bInteger ? new CPDF_Number(m_Integer) : new CPDF_Number(m_Float);
 }
 
 FX_FLOAT CPDF_Number::GetNumber() const {
   return m_bInteger ? static_cast<FX_FLOAT>(m_Integer) : m_Float;
 }
 
 int CPDF_Number::GetInteger() const {
   return m_bInteger ? m_Integer : static_cast<int>(m_Float);
 }
@@ skipping 11 matching lines @@
 }
 
 void CPDF_Number::SetString(const CFX_ByteString& str) {
   m_bInteger = FX_atonum(str.AsStringC(), &m_Integer);
 }
 
 CFX_ByteString CPDF_Number::GetString() const {
   return m_bInteger ? CFX_ByteString::FormatInteger(m_Integer, FXFORMAT_SIGNED)
                     : CFX_ByteString::FormatFloat(m_Float);
 }