From 2f38cad98747095cabdd6bdb523316d736655d9c Mon Sep 17 00:00:00 2001
From: Ben Wiederhake <BenWiederhake.GitHub@gmx.de>
Date: Tue, 7 Sep 2021 21:35:36 +0200
Subject: [PATCH] Assistant: Fix lockfile logic

Previously, Assistant was able to re-execute itself due to a quirk in
the global cursor tracking logic, it seems.

Now it has become necessary to explicitly drop the lockfile before
activating the new Assistant instance.

Alternatively, this has always been a latent bug that only now has
become more likely.
---
 Userland/Applications/Assistant/main.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Userland/Applications/Assistant/main.cpp b/Userland/Applications/Assistant/main.cpp
index 9bff51af66..426d8913ac 100644
--- a/Userland/Applications/Assistant/main.cpp
+++ b/Userland/Applications/Assistant/main.cpp
@@ -244,6 +244,7 @@ int main(int argc, char** argv)
     text_box.on_return_pressed = [&]() {
         if (!app_state.selected_index.has_value())
             return;
+        lockfile.release();
         app_state.results[app_state.selected_index.value()].activate();
         GUI::Application::the()->quit();
     };