Fix errors from commit f0a169e.
authorLei Zhang <thestig@chromium.org>
Fri, 8 May 2015 18:33:26 +0000 (11:33 -0700)
committerLei Zhang <thestig@chromium.org>
Fri, 8 May 2015 18:33:26 +0000 (11:33 -0700)
- fread() returns the number of items read.
- fix a memory leak in error handling.

R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1135713004

core/src/fxge/ge/fx_ge_fontmap.cpp

index 0596c12..f058f03 100644 (file)
@@ -1367,7 +1367,7 @@ void CFX_FolderFontInfo::ScanFile(CFX_ByteString& path)
     FX_BYTE buffer[16];
     FXSYS_fseek(pFile, 0, FXSYS_SEEK_SET);
     size_t readCnt = FXSYS_fread(buffer, 12, 1, pFile);
-    if (readCnt != 12) {
+    if (readCnt != 1) {
         FXSYS_fclose(pFile);
         return;
     }
@@ -1386,6 +1386,7 @@ void CFX_FolderFontInfo::ScanFile(CFX_ByteString& path)
         }
         readCnt = FXSYS_fread(offsets, face_bytes, 1, pFile);
         if (readCnt != face_bytes) {
+            FX_Free(offsets);
             FXSYS_fclose(pFile);
             return;
         }