From: Behdad Esfahbod <behdad@behdad.org>
Date: Thu, 30 Aug 2018 01:13:07 +0000 (-0700)
Subject: [subset] Bail if subsetting a table failed
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d237ce71810045ee400c52740cd8ab6e1113aada;p=platform%2Fupstream%2FlibHarfBuzzSharp.git

[subset] Bail if subsetting a table failed
---

diff --git a/src/hb-subset.cc b/src/hb-subset.cc
index 1695d37..a072997 100644
--- a/src/hb-subset.cc
+++ b/src/hb-subset.cc
@@ -53,11 +53,9 @@ _subset (hb_subset_plan_t *plan)
   hb_tag_t tag = TableType::tableTag;
   hb_bool_t result = false;
   if (source_blob->data)
-  {
-    result = table->subset(plan);
-  } else {
+    result = table->subset (plan);
+  else
     DEBUG_MSG(SUBSET, nullptr, "OT::%c%c%c%c::subset sanitize failed on source table.", HB_UNTAG(tag));
-  }
 
   hb_blob_destroy (source_blob);
   DEBUG_MSG(SUBSET, nullptr, "OT::%c%c%c%c::subset %s", HB_UNTAG(tag), result ? "success" : "FAILED!");
@@ -196,7 +194,7 @@ hb_subset (hb_face_t *source,
       success = success && _subset_table (plan, tag);
     }
     offset += count;
-  } while (count == ARRAY_LENGTH (table_tags));
+  } while (success && count == ARRAY_LENGTH (table_tags));
 
   hb_face_t *result = success ? hb_face_reference(plan->dest) : hb_face_get_empty();
   hb_subset_plan_destroy (plan);