- drop permissions check in GetPluginsReport and ClearPluginsReport

diff --git a/package/snapper.changes b/package/snapper.changes
index 3b53926ba1f3853211ccc052fcd4cfa24a6fbe30..1e05f94b4e8f6c6a1ff94268779c2f046acf88dd 100644 (file)
--- a/package/snapper.changes
+++ b/package/snapper.changes
@@ -1,3 +1,9 @@
+-------------------------------------------------------------------
+Fri Dec 08 09:06:30 CET 2023 - aschnell@suse.com
+
+- drop permissions check in GetPluginsReport and ClearPluginsReport
+  (gh#openSUSE/snapper#856)
+
 -------------------------------------------------------------------
 Tue Nov 28 08:22:41 CET 2023 - aschnell@suse.com
 

diff --git a/server/Client.cc b/server/Client.cc
index 68f89d793eacf0a093ec4d3557a2248b2761eb9b..0309d24e73cf02929cd68d82c17fb375ddafc629 100644 (file)
--- a/server/Client.cc
+++ b/server/Client.cc
@@ -1738,7 +1738,7 @@ Client::get_plugins_report(DBus::Connection& conn, DBus::Message& msg)
 
     boost::unique_lock<boost::shared_mutex> lock(big_mutex);
 
-    check_permission(conn, msg);
+    // No permission check here: The report belongs to the client.
 
     DBus::MessageMethodReturn reply(msg);
 
@@ -1756,7 +1756,7 @@ Client::clear_plugins_report(DBus::Connection& conn, DBus::Message& msg)
 
     boost::unique_lock<boost::shared_mutex> lock(big_mutex);
 
-    check_permission(conn, msg);
+    // No permission check here: The report belongs to the client.
 
     DBus::MessageMethodReturn reply(msg);