Manually drop records if Sorter is dropped early (e.g. in case of error)

Author: jussisaurio

core/vdbe/sorter.rs

@@ -408,6 +408,18 @@ impl Sorter {
     }
 }
 
+impl Drop for Sorter {
+    fn drop(&mut self) {
+        // Drop all arena-allocated records to free their inner heap allocations
+        // (Vec, ImmutableRecord payload, etc.) before the arena memory is released.
+        // Bumpalo doesn't run destructors, so we must do this manually.
+        for ptr in self.records.drain(..) {
+            // SAFETY: All pointers are valid (arena not yet dropped).
+            unsafe { std::ptr::drop_in_place(ptr.as_ptr()) };
+        }
+    }
+}
+
 #[derive(Debug, Clone, Copy)]
 enum NextState {
     Start,