The provider can be unreffed in vpn_died() or in vpn_disconnect().
We must do the unref only once.
---
 plugins/vpn.c |    8 ++++++--
 1 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/plugins/vpn.c b/plugins/vpn.c
index 7a35f7a..e41a374 100644
--- a/plugins/vpn.c
+++ b/plugins/vpn.c
@@ -134,8 +134,12 @@ void vpn_died(struct connman_task *task, int exit_code, 
void *user_data)
 
        stop_vpn(provider);
        connman_provider_set_data(provider, NULL);
-       connman_provider_unref(provider);
-       connman_rtnl_remove_watch(data->watch);
+
+       if (data->watch != 0) {
+               connman_provider_unref(provider);
+               connman_rtnl_remove_watch(data->watch);
+               data->watch = 0;
+       }
 
 vpn_exit:
        if (state != VPN_STATE_READY && state != VPN_STATE_DISCONNECT) {
-- 
1.7.5.4

_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to