Hi, this patchset implements multiple agent support. This means that multiple agents can be registered and connman directs user requests to best agent available.
In this v2 patchset, the patch 5 is split into four pieces. Patch 1 contains no functionality changes and is just a preparation for the rest of the stuff. Patch 2 checks that we do not try to pass NULL pointer to DBus API as that will cause crash in DBus library. Patches 3 and 4 cleanup all the pending agent requests in service and vpn provider before service and provider hashes are cleared. This prevents free memory access that would happen otherwise. Patch 5 contains the agent API change which is needed by the actual multi agent support in patch 6. Patch 5 does not contain any functionality changes. Patch 7 and 8 add checks that there is an agent before trying to ask user. Cheers, Jukka Jukka Rissanen (8): agent: Change the name of the agent struct agent: Check that message exists before calling dbus service: Unregister agent driver before removing service vpn: Unregister agent driver before removing provider agent: API changes required for multi agent support agent: Add multi agent support vpn: Make sure we got an agent before asking user agent: Make sure we got an agent before asking user include/agent.h | 9 +- src/agent-connman.c | 32 +++- src/agent.c | 468 +++++++++++++++++++++++++++++++--------------- src/connman.h | 3 +- src/service.c | 20 +- vpn/plugins/l2tp.c | 18 +- vpn/plugins/openconnect.c | 17 +- vpn/plugins/openvpn.c | 3 +- vpn/plugins/pptp.c | 18 +- vpn/plugins/vpn.c | 6 +- vpn/plugins/vpn.h | 3 +- vpn/plugins/vpnc.c | 3 +- vpn/vpn-provider.c | 8 +- vpn/vpn-provider.h | 3 +- 14 files changed, 408 insertions(+), 203 deletions(-) -- 1.7.11.7 _______________________________________________ connman mailing list [email protected] https://lists.connman.net/mailman/listinfo/connman
