Merge pull request #4202 from randombit/fix/scoped_cleanup

FIX: move() of scoped_cleanup
randombit · Jul 12, 2024 · d88042e · d88042e
2 parents 2bc1985 + 4e05de7
commit d88042e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/src/lib/utils/stl_util.h b/src/lib/utils/stl_util.h
@@ -357,8 +357,16 @@ class scoped_cleanup {
 
       scoped_cleanup(const scoped_cleanup&) = delete;
       scoped_cleanup& operator=(const scoped_cleanup&) = delete;
-      scoped_cleanup(scoped_cleanup&&) = default;
-      scoped_cleanup& operator=(scoped_cleanup&&) = default;
+
+      scoped_cleanup(scoped_cleanup&& other) : m_cleanup(std::move(other.m_cleanup)) { other.disengage(); }
+
+      scoped_cleanup& operator=(scoped_cleanup&& other) {
+         if(this != &other) {
+            m_cleanup = std::move(other.m_cleanup);
+            other.disengage();
+         }
+         return *this;
+      }
 
       ~scoped_cleanup() {
          if(m_cleanup.has_value()) {

diff --git a/src/tests/test_utils.cpp b/src/tests/test_utils.cpp
@@ -1232,6 +1232,8 @@ class UUID_Tests : public Test {
 
 BOTAN_REGISTER_TEST("utils", "uuid", UUID_Tests);
 
+#endif
+
 class Formatter_Tests : public Test {
    public:
       std::vector<Test::Result> run() override {
@@ -1323,8 +1325,6 @@ class ScopedCleanup_Tests : public Test {
 
 BOTAN_REGISTER_TEST("utils", "scoped_cleanup", ScopedCleanup_Tests);
 
-#endif
-
 }  // namespace
 
 }  // namespace Botan_Tests