agent-manager: let nm_settings_connection_check_permission() check all secret-agents searching for permission

nm_agent_manager_get_agent_by_user() would only return the first
matching secret agent for the user. This way, we might miss an agent
that has permissions.

Instead, add nm_agent_manager_has_agent_with_permission() and search
all agents.

1 files changed, 20 insertions, 20 deletions

diff --git a/src/settings/nm-agent-manager.c b/src/settings/nm-agent-manager.c
index c5a0d9dea1..7c45bc8f79 100644
--- a/src/settings/nm-agent-manager.c
+++ b/src/settings/nm-agent-manager.c
@@ -226,21 +226,6 @@ _agent_find_by_identifier_and_uid (NMAgentManagerPrivate *priv,
 	return NULL;
 }
 
-static NMSecretAgent *
-_agent_find_by_username (NMAgentManagerPrivate *priv,
-                         const char *username)
-{
-	NMSecretAgent *agent;
-
-	nm_assert (username);
-
-	c_list_for_each_entry (agent, &priv->agent_lst_head, agent_lst) {
-		if (nm_streq0 (nm_secret_agent_get_owner_username (agent), username))
-			return agent;
-	}
-	return NULL;
-}
-
 /*****************************************************************************/
 
 static void
@@ -1402,13 +1387,28 @@ nm_agent_manager_delete_secrets (NMAgentManager *self,
 
 /*****************************************************************************/
 
-NMSecretAgent *
-nm_agent_manager_get_agent_by_user (NMAgentManager *self, const char *username)
+gboolean
+nm_agent_manager_has_agent_with_permission (NMAgentManager *self,
+                                            const char *username,
+                                            const char *permission)
 {
-	g_return_val_if_fail (NM_IS_AGENT_MANAGER (self), NULL);
-	g_return_val_if_fail (username, NULL);
+	NMAgentManagerPrivate *priv;
+	NMSecretAgent *agent;
+
+	g_return_val_if_fail (NM_IS_AGENT_MANAGER (self), FALSE);
+	g_return_val_if_fail (username, FALSE);
+	g_return_val_if_fail (permission, FALSE);
+
+	priv = NM_AGENT_MANAGER_GET_PRIVATE (self);
 
-	return _agent_find_by_username (NM_AGENT_MANAGER_GET_PRIVATE (self), username);
+	c_list_for_each_entry (agent, &priv->agent_lst_head, agent_lst) {
+		if (!nm_streq0 (nm_secret_agent_get_owner_username (agent), username))
+			continue;
+		if (nm_secret_agent_has_permission (agent, permission))
+			return TRUE;
+	}
+
+	return FALSE;
 }
 
 /*****************************************************************************/