Additional non-nullptr protection

single-cell-data · Mar 28, 2024 · 2a27207 · 2a27207
1 parent 53b9e57
commit 2a27207
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/libtiledbsoma/src/utils/arrow_adapter.cc b/libtiledbsoma/src/utils/arrow_adapter.cc
@@ -95,15 +95,17 @@ void ArrowAdapter::release_schema(struct ArrowSchema* schema) {
 
 void ArrowAdapter::release_array(struct ArrowArray* array) {
     auto arrow_buffer = static_cast<ArrowBuffer*>(array->private_data);
-    LOG_TRACE(fmt::format(
-        "[ArrowAdapter] release_array {} use_count={}",
-        arrow_buffer->buffer_->name(),
-        arrow_buffer->buffer_.use_count()));
-
-    // Delete the ArrowBuffer, which was allocated with new.
-    // If the ArrowBuffer.buffer_ shared_ptr is the last reference to the
-    // underlying ColumnBuffer, the ColumnBuffer will be deleted.
-    delete arrow_buffer;
+    if (arrow_buffer != nullptr) {
+      LOG_TRACE(fmt::format(
+          "[ArrowAdapter] release_array {} use_count={}",
+          arrow_buffer->buffer_->name(),
+          arrow_buffer->buffer_.use_count()));
+
+      // Delete the ArrowBuffer, which was allocated with new.
+      // If the ArrowBuffer.buffer_ shared_ptr is the last reference to the
+      // underlying ColumnBuffer, the ColumnBuffer will be deleted.
+      delete arrow_buffer;
+    }
 
     if (array->buffers != nullptr) {
         free(array->buffers);