Moved deallocation of mem block device to destructor

Enables persistant storage between de/reinitialization cycles

Author: geky

features/filesystem/bd/MemBlockDevice.cpp

@@ -26,32 +26,34 @@ MemBlockDevice::MemBlockDevice(bd_count_t count, bd_size_t size)
 
 MemBlockDevice::~MemBlockDevice()
 {
-    // nop if uninitialized
-    deinit();
+    if (_blocks) {
+        for (bd_count_t i = 0; i < _count; i++) {
+            free(_blocks[i]);
+        }
+
+        free(_blocks);
+        _blocks = 0;
+    }
 }
 
 bd_error_t MemBlockDevice::init()
 {
-    _blocks = (uint8_t**)malloc(_count*sizeof(uint8_t*));
     if (!_blocks) {
-        return BD_ERROR_DEVICE_ERROR;
+        _blocks = (uint8_t**)malloc(_count*sizeof(uint8_t*));
+        if (!_blocks) {
+            return BD_ERROR_DEVICE_ERROR;
+        }
+
+        memset(_blocks, 0, _count*sizeof(uint8_t*));
     }
 
-    memset(_blocks, 0, _count*sizeof(uint8_t*));
     return BD_ERROR_OK;
 }
 
 bd_error_t MemBlockDevice::deinit()
 {
-    if (_blocks) {
-        for (bd_count_t i = 0; i < _count; i++) {
-            free(_blocks[i]);
-        }
-
-        free(_blocks);
-        _blocks = 0;
-    }
-
+    // Memory is lazily cleaned up in destructor to allow
+    // data to live across de/reinitialization
     return BD_ERROR_OK;
 }