[gnome-shell] gdm: Increase the verification failed counter once we've a failure



commit 1158e989134d61f943a6af3828fc0227cc0709c4
Author: Marco Trevisan (TreviƱo) <mail 3v1n0 net>
Date:   Mon Feb 8 18:43:16 2021 +0100

    gdm: Increase the verification failed counter once we've a failure
    
    Decouple the verification failure count increase from
    _verificationFailed as there are some cases in which we may want to
    increase it without emitting a verification-failed signal.
    
    Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1652>

 js/gdm/util.js | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
---
diff --git a/js/gdm/util.js b/js/gdm/util.js
index 969e126682..0a275d50e2 100644
--- a/js/gdm/util.js
+++ b/js/gdm/util.js
@@ -384,6 +384,7 @@ var ShellUserVerifier = class {
         this._hold.release();
 
         this._queueMessage(_("Authentication error"), MessageType.ERROR);
+        this._failCounter++;
         this._verificationFailed(serviceName, false);
     }
 
@@ -609,7 +610,6 @@ var ShellUserVerifier = class {
         // Otherwise, when in login mode we allow ALLOWED_FAILURES attempts.
         // After that, we go back to the welcome screen.
 
-        this._failCounter++;
         let canRetry = retry && this._userName &&
             (this._reauthOnly ||
              this._failCounter < this._settings.get_int(ALLOWED_FAILURES_KEY));
@@ -660,8 +660,10 @@ var ShellUserVerifier = class {
         // if the password service fails, then cancel everything.
         // But if, e.g., fingerprint fails, still give
         // password authentication a chance to succeed
-        if (this.serviceIsForeground(serviceName))
+        if (this.serviceIsForeground(serviceName)) {
+            this._failCounter++;
             this._verificationFailed(serviceName, true);
+        }
     }
 };
 Signals.addSignalMethods(ShellUserVerifier.prototype);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]