From: JUN FANG Date: Mon, 13 Jul 2015 11:20:10 +0000 (-0700) Subject: Fix a crashier due to incorrect type conversion X-Git-Url: http://downloads.foxitsoftware.com/web/?p=pdfium.git;a=commitdiff_plain;h=1f4c2f24709e0246575551cecdaa0ba83de73101;hp=aa575d4805bf7a9e83f00b56e38d4de39d90f95c Fix a crashier due to incorrect type conversion BUG=issue179 R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1217093006 . --- diff --git a/core/src/fxge/android/fpf_skiafont.cpp b/core/src/fxge/android/fpf_skiafont.cpp index 40b1886..2dad010 100644 --- a/core/src/fxge/android/fpf_skiafont.cpp +++ b/core/src/fxge/android/fpf_skiafont.cpp @@ -159,12 +159,13 @@ int32_t CFPF_SkiaFont::GetItalicAngle() const FX_DWORD CFPF_SkiaFont::GetFontData(FX_DWORD dwTable, uint8_t* pBuffer, FX_DWORD dwSize) { if (!m_Face) { - return FALSE; + return 0; } - if (FXFT_Load_Sfnt_Table(m_Face, dwTable, 0, pBuffer, (unsigned long*)&dwSize)) { + FT_ULong ulSize = pdfium::base::checked_cast(dwSize); + if (FXFT_Load_Sfnt_Table(m_Face, dwTable, 0, pBuffer, &ulSize)) { return 0; } - return dwSize; + return pdfium::base::checked_cast(ulSize); } FX_BOOL CFPF_SkiaFont::InitFont(CFPF_SkiaFontMgr *pFontMgr, CFPF_SkiaFontDescriptor *pFontDes, const CFX_ByteStringC& bsFamily, FX_DWORD dwStyle, uint8_t uCharset) {