From 80a7a8ddb9cfd54cab091057e022f7a406313855 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marco=20Trevisan=20=28Trevi=C3=B1o=29?= <mail@3v1n0.net>
Date: Tue, 2 Feb 2021 17:29:29 +0100
Subject: [PATCH] gdm: Ensure we cancel all the previously running services on
 auth retry

When retrying the authentication we should make sure that all the
previously initiated services are stopped in order to begin a new
authentication session with all the configured services.

Unfortunately at the current state we only dispose the currently used
user verifier, but we don't make it to stop all the relative gdm workers
and then they'll stay around potentially blocking any further usage of
them (as it happens with the fingerprint one, that has unique access to
the device).

So, cancel the currently running authentication before starting a new
one if we're explicitly retrying.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1622>
---
 js/gdm/util.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/js/gdm/util.js b/js/gdm/util.js
index a35ab94b9..320d2f826 100644
--- a/js/gdm/util.js
+++ b/js/gdm/util.js
@@ -591,6 +591,7 @@ var ShellUserVerifier = class {
     }
 
     _retry() {
+        this.cancel();
         this.begin(this._userName, new Batch.Hold());
     }