commit 47cd7996cc6d3393e3fc95fb7b141833d0e167b7
author John Wedig <johnwedig@google.com> Wed Oct 05 15:45:11 2022 -0700
committer John Wedig <johnwedig@google.com> Wed Oct 05 15:55:39 2022 -0700
tree 805b9ba85a352c1cffff9913e668d09dd1b2f34c
parent 8d5a3a0751b7d47cb01cff25449e83e4a633aa4f

Fix logic error for SecuredLocked EraseMethod

The handling for the SecuredLocked EraseMethod is missing a '!'
operator. We want to lock the LUKS volume if it is NOT already locked.

Tested:
Loaded eStoraged on a machine and exercised the SecuredLocked code path
to confirm the expected behavior, i.e. lock if it's not already lock.

Signed-off-by: John Wedig <johnwedig@google.com>
Change-Id: Iac55e56092cbef45e6212e1fa3b7a60ef8e2a60f

src/estoraged.cpp

diff --git a/src/estoraged.cpp b/src/estoraged.cpp
index ef3bd63..5da6763 100644
--- a/src/estoraged.cpp
+++ b/src/estoraged.cpp
@@ -205,7 +205,7 @@
         }
         case Volume::EraseMethod::SecuredLocked:
         {
-            if (isLocked())
+            if (!isLocked())
             {
                 lock();
             }