Only call std::allocator::deallocate() in FixedSizeVector for non-empty vectors. When using a malloc-based std:allocator this was not an issue (since free(0) is a no-op), but it can cause problems with other STL implementations.

1 files changed, 3 insertions, 1 deletions

diff --git a/include/fruit/impl/data_structures/fixed_size_vector.defn.h b/include/fruit/impl/data_structures/fixed_size_vector.defn.h
index 3573181..d5d36f9 100644
--- a/include/fruit/impl/data_structures/fixed_size_vector.defn.h
+++ b/include/fruit/impl/data_structures/fixed_size_vector.defn.h
@@ -42,7 +42,9 @@ inline FixedSizeVector<T, Allocator>::FixedSizeVector(std::size_t capacity, Allo
 template <typename T, typename Allocator>
 inline FixedSizeVector<T, Allocator>::~FixedSizeVector() {
   clear();
-  allocator.deallocate(v_begin, capacity);
+  if (capacity != 0) {
+    allocator.deallocate(v_begin, capacity);
+  }
 }
 
 template <typename T, typename Allocator>