[devicelock] Do not check code twice. Contributes to JB#45370

Due to a bug this checked security code twice when authentication was
successful. Instead of calling both checkCode and unlockWithCode it
should be calling only unlockWithCode and which itself is calling
checkCode.

Signed-off-by: Tomi Leppänen <tomi.leppanen@jolla.com>

src/nemo-devicelock/host/hostdevicelock.cpp

@@ -148,9 +148,10 @@ void HostDeviceLock::enterSecurityCode(const QString &code)
     case Idle:
         break;
     case Authenticating: {
-        switch (const int result = checkCode(code)) {
+        switch (const int result = unlockWithCode(code)) {
+        case Evaluating:
         case Success:
-            unlockFinished(unlockWithCode(code), Authenticator::SecurityCode);
+            unlockFinished(result, Authenticator::SecurityCode);
             break;
         case SecurityCodeExpired:
             m_state = EnteringNewSecurityCode;