Re-sync with internal repository (facebook#791)

Co-authored-by: Facebook Community Bot <[email protected]>
hee-develop · Aug 2, 2022 · 5b6255a · 5b6255a
1 parent 8984dc9
commit 5b6255a
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/lib/VM/detail/IdentifierHashTable.cpp b/lib/VM/detail/IdentifierHashTable.cpp
@@ -138,6 +138,10 @@ void IdentifierHashTable::remove(const StringPrimitive *str) {
 }
 
 void IdentifierHashTable::growAndRehash(uint32_t newCapacity) {
+  // Guard against potential overflow in the calculation of new capacity.
+  if (LLVM_UNLIKELY(newCapacity <= capacity())) {
+    hermes_fatal("too many identifiers created");
+  }
   assert(llvh::isPowerOf2_32(newCapacity) && "capacity must be power of 2");
   CompactTable tmpTable(newCapacity, table_.getCurrentScale());
   tmpTable.swap(table_);