[HB] Fix a couple other sanitize() bugs
authorBehdad Esfahbod <behdad@behdad.org>
Wed, 5 Aug 2009 03:24:35 +0000 (23:24 -0400)
committerBehdad Esfahbod <behdad@behdad.org>
Mon, 2 Nov 2009 19:40:31 +0000 (14:40 -0500)
src/hb-ot-layout-gpos-private.hh
src/hb-ot-layout-gsub-private.hh

index bbf2773..70dc791 100644 (file)
@@ -1497,7 +1497,7 @@ struct PosLookup : Lookup
 
   inline bool sanitize (SANITIZE_ARG_DEF) {
     SANITIZE_DEBUG ();
-    if (Lookup::sanitize (SANITIZE_ARG)) return false;
+    if (!Lookup::sanitize (SANITIZE_ARG)) return false;
     OffsetArrayOf<PosLookupSubTable> &list = (OffsetArrayOf<PosLookupSubTable> &) subTable;
     return SANITIZE_THIS (list);
   }
@@ -1529,7 +1529,7 @@ struct GPOS : GSUBGPOS
 
   bool sanitize (SANITIZE_ARG_DEF) {
     SANITIZE_DEBUG ();
-    if (GSUBGPOS::sanitize (SANITIZE_ARG)) return false;
+    if (!GSUBGPOS::sanitize (SANITIZE_ARG)) return false;
     OffsetTo<PosLookupList> &list = CAST(OffsetTo<PosLookupList>, lookupList, 0);
     return SANITIZE_THIS (list);
   }
index 7b421db..a46006d 100644 (file)
@@ -822,7 +822,7 @@ struct SubstLookup : Lookup
 
   inline bool sanitize (SANITIZE_ARG_DEF) {
     SANITIZE_DEBUG ();
-    if (Lookup::sanitize (SANITIZE_ARG)) return false;
+    if (!Lookup::sanitize (SANITIZE_ARG)) return false;
     OffsetArrayOf<SubstLookupSubTable> &list = (OffsetArrayOf<SubstLookupSubTable> &) subTable;
     return SANITIZE_THIS (list);
   }
@@ -855,7 +855,7 @@ struct GSUB : GSUBGPOS
 
   bool sanitize (SANITIZE_ARG_DEF) {
     SANITIZE_DEBUG ();
-    if (GSUBGPOS::sanitize (SANITIZE_ARG)) return false;
+    if (!GSUBGPOS::sanitize (SANITIZE_ARG)) return false;
     OffsetTo<SubstLookupList> &list = CAST(OffsetTo<SubstLookupList>, lookupList, 0);
     return SANITIZE_THIS (list);
   }