Fixed a potential bug that could cause a Segment Fault when insert a …

…large tuple
cmu-db · Jun 25, 2024 · ecb0643 · ecb0643
1 parent 70bcc50
commit ecb0643
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/storage/page/table_page.cpp b/src/storage/page/table_page.cpp
@@ -38,7 +38,7 @@ auto TablePage::GetNextTupleOffset(const TupleMeta &meta, const Tuple &tuple) co
   }
   auto tuple_offset = slot_end_offset - tuple.GetLength();
   auto offset_size = TABLE_PAGE_HEADER_SIZE + TUPLE_INFO_SIZE * (num_tuples_ + 1);
-  if (tuple_offset < offset_size) {
+  if (tuple_offset < offset_size || tuple_offset >= BUSTUB_PAGE_SIZE) {
     return std::nullopt;
   }
   return tuple_offset;