ringbuffer: Fix cloneFrom(), resetImpl(); Validate dtor array nullptr (ABORT)

- cloneFrom(): Used wrong arguments to freeArray
- resetImpl(): For public reset(..): arrays wasn't freed when resizing

1 files changed, 7 insertions, 1 deletions

diff --git a/include/jau/ringbuffer.hpp b/include/jau/ringbuffer.hpp
index f9ade91..2b7a7ba 100644
--- a/include/jau/ringbuffer.hpp
+++ b/include/jau/ringbuffer.hpp
@@ -116,7 +116,7 @@ template <typename T, std::nullptr_t nullelem, typename Size_type> class ringbuf
             if( allocArrayAndCapacity ) {
                 capacityPlusOne = source.capacityPlusOne;
                 if( nullptr != array ) {
-                    freeArray(array, true);
+                    freeArray(array);
                 }
                 array = newArray(capacityPlusOne);
             } else if( capacityPlusOne != source.capacityPlusOne ) {
@@ -161,6 +161,9 @@ template <typename T, std::nullptr_t nullelem, typename Size_type> class ringbuf
             if( nullptr != copyFrom && 0 < copyFromCount ) {
                 if( copyFromCount > capacityPlusOne-1 ) {
                     // new blank resized array
+                    if( nullptr != array ) {
+                        freeArray(array);
+                    }
                     capacityPlusOne = copyFromCount + 1;
                     array = newArray(capacityPlusOne);
                     readPos = 0;
@@ -408,6 +411,9 @@ template <typename T, std::nullptr_t nullelem, typename Size_type> class ringbuf
         { }
 
         ~ringbuffer() noexcept {
+            if( nullptr == array ) {
+                ABORT("ringbuffer::dtor array==nullptr");
+            }
             freeArray(array);
         }