From de6f7c87f098009dcd6e34a779f6e373a1c9f5c7 Mon Sep 17 00:00:00 2001
From: Ivailo Monev <xakepa10@gmail.com>
Date: Wed, 30 Nov 2022 03:03:38 +0200
Subject: [PATCH] plasma: check the reply of the
 org.freedesktop.ScreenSaver.Lock call in PowerManagementJob::start()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
---
 plasma/dataengines/powermanagement/powermanagementjob.cpp | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/plasma/dataengines/powermanagement/powermanagementjob.cpp b/plasma/dataengines/powermanagement/powermanagementjob.cpp
index ba8c4bb6..c528057c 100644
--- a/plasma/dataengines/powermanagement/powermanagementjob.cpp
+++ b/plasma/dataengines/powermanagement/powermanagementjob.cpp
@@ -47,10 +47,13 @@ void PowerManagementJob::start()
     //kDebug() << "starting operation  ... " << operation;
 
     if (operation == "lockScreen") {
-#warning TODO: error check
         static const QString interface("org.freedesktop.ScreenSaver");
         QDBusInterface screensaver(interface, "/ScreenSaver");
-        screensaver.asyncCall("Lock");
+        QDBusPendingReply<void> reply = screensaver.asyncCall("Lock");
+        if (reply.isError()) {
+            setResult(false);
+            return;
+        }
         setResult(true);
         return;
     } else if (operation == "suspend" || operation == "suspendToRam") {