memorypool: placed mutex around get and release functions

Makes all memory pool derivatives effectively thread-safe.

app/common/memorypool.h

@@ -23,10 +23,9 @@
 
 #include <memory>
 #include <QDateTime>
+#include <QMutex>
 #include <stdint.h>
 
-#include <QDebug>
-
 #include "common/define.h"
 
 OLIVE_NAMESPACE_ENTER
@@ -39,7 +38,7 @@ public:
     data_ = nullptr;
   }
 
-  ~MemoryPool() {
+  virtual ~MemoryPool() {
     delete [] data_;
   }
 
@@ -67,6 +66,8 @@ public:
   }
 
   void Destroy() {
+    // FIXME: Invalidate elements sent out here?
+
     delete [] data_;
     data_ = nullptr;
 
@@ -127,6 +128,8 @@ public:
   using ElementPtr = std::shared_ptr<Element>;
 
   ElementPtr Get() {
+    QMutexLocker locker(&lock_);
+
     for (int i=0;i<available_.size();i++) {
       if (available_.at(i)) {
         // This buffer is available
@@ -141,6 +144,7 @@ public:
   }
 
   void Release(Element* e) {
+    QMutexLocker locker(&lock_);
     quintptr diff = reinterpret_cast<quintptr>(e->data()) - reinterpret_cast<quintptr>(data_);
 
     int index = diff / GetElementSize();
@@ -158,6 +162,8 @@ private:
 
   QVector<bool> available_;
 
+  QMutex lock_;
+
 };
 
 OLIVE_NAMESPACE_EXIT