client: Rework agent registarion logic

Do agent registration as part of agent manager proxy creation.
This ensures that the registration call is made only after the agent
manager’s interface becomes available on the bus.
This commit is contained in:
Tim Kourt 2020-02-26 15:32:24 -08:00 committed by Denis Kenzior
parent 4fdb6bc9fa
commit df64dd443e
3 changed files with 22 additions and 23 deletions

View File

@ -2,7 +2,7 @@
*
* Wireless daemon for Linux
*
* Copyright (C) 2017-2019 Intel Corporation. All rights reserved.
* Copyright (C) 2017-2020 Intel Corporation. All rights reserved.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
@ -30,6 +30,7 @@
#include "client/agent.h"
#include "client/dbus-proxy.h"
#include "client/agent-manager.h"
#include "client/command.h"
#define IWD_AGENT_MANAGER_PATH "/net/connman/iwd"
@ -39,18 +40,16 @@ static void check_errors_method_callback(struct l_dbus_message *message,
dbus_message_has_error(message);
}
bool agent_manager_register_agent(void)
static bool agent_manager_register_agent(const char *path)
{
const char *path;
const struct proxy_interface *proxy =
proxy_interface_find(IWD_AGENT_MANAGER_INTERFACE,
IWD_AGENT_MANAGER_PATH);
const struct proxy_interface *proxy;
if (!proxy)
if (!path)
return false;
path = proxy_interface_get_data(proxy);
if (!path)
proxy = proxy_interface_find(IWD_AGENT_MANAGER_INTERFACE,
IWD_AGENT_MANAGER_PATH);
if (!proxy)
return false;
proxy_interface_method_call(proxy, "RegisterAgent", "o",
@ -81,10 +80,17 @@ bool agent_manager_unregister_agent(void)
static void *agent_manager_create(void)
{
char *path = l_strdup_printf("/agent/%i", getpid());
char *path;
if (command_needs_no_agent())
return NULL;
path = l_strdup_printf("/agent/%i", getpid());
agent_init(path);
agent_manager_register_agent(path);
return path;
}
@ -92,6 +98,9 @@ static void agent_manager_destroy(void *data)
{
char *path = data;
if (!path)
return;
agent_exit(path);
l_free(path);

View File

@ -2,7 +2,7 @@
*
* Wireless daemon for Linux
*
* Copyright (C) 2017-2019 Intel Corporation. All rights reserved.
* Copyright (C) 2017-2020 Intel Corporation. All rights reserved.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
@ -20,5 +20,4 @@
*
*/
bool agent_manager_register_agent(void);
bool agent_manager_unregister_agent(void);

View File

@ -2,7 +2,7 @@
*
* Wireless daemon for Linux
*
* Copyright (C) 2017-2019 Intel Corporation. All rights reserved.
* Copyright (C) 2017-2020 Intel Corporation. All rights reserved.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
@ -745,14 +745,6 @@ static void get_managed_objects_callback(struct l_dbus_message *message,
while (l_dbus_message_iter_next_entry(&objects, &path, &object))
proxy_interfaces_update_properties(path, &object);
if (!command_needs_no_agent()) {
if (!agent_manager_register_agent()) {
display_error("Failed to register Agent.\n");
goto error;
}
}
if (command_is_interactive_mode())
display_enable_cmd_prompt();
else
@ -878,8 +870,7 @@ bool dbus_proxy_exit(void)
{
struct interface_type_desc *desc;
if (!command_needs_no_agent())
agent_manager_unregister_agent();
agent_manager_unregister_agent();
for (desc = __start___interface; desc < __stop___interface; desc++) {
if (!desc->exit)