commit 30e405e470f002693b353db5a1bb90504ba01b2a
author Garret Rieger <grieger@google.com> Thu Dec 01 22:12:59 2022 +0000
committer Garret Rieger <grieger@google.com> Thu Dec 01 22:12:59 2022 +0000
tree 52aac26ef736d5ccc01940cadb98bd4cbb461c4f
parent 554ed06fac759508ad959482f784bf02e4839a66

[repacker] ensure link obj indices are valid.

src/graph/graph.hh

diff --git a/src/graph/graph.hh b/src/graph/graph.hh
index 66c07e5..aac2fbe 100644
--- a/src/graph/graph.hh
+++ b/src/graph/graph.hh
@@ -50,11 +50,18 @@
     unsigned priority = 0;
 
 
-    bool link_positions_valid ()
+    bool link_positions_valid (unsigned num_objects)
     {
       hb_set_t assigned_bytes;
       for (const auto& l : obj.real_links)
       {
+        if (l.objidx >= num_objects)
+        {
+          DEBUG_MSG (SUBSET_REPACK, nullptr,
+                     "Invalid graph. Invalid object index.");
+          return false;
+        }
+
         unsigned start = l.position;
         unsigned end = start + l.width - 1;
 
@@ -333,7 +340,7 @@
       if (check_success (!vertices_.in_error ()))
         v->obj = *objects[i];
 
-      check_success (v->link_positions_valid ());
+      check_success (v->link_positions_valid (objects.length));
 
       if (!removed_nil) continue;
       // Fix indices to account for removed nil object.