vpn: Clear the data pointer from provider when destroying

If the data pointer is left around, then vpn plugin might
accidentally use it if dbus messages are received after
we have cleared the vpn connection.

diff --git a/plugins/vpn.c b/plugins/vpn.c
index cbba39655cf278b95e2fe49476f24291006a5cd0..f60a658b095fdf263a587cfe79cac93d94424ad6 100644 (file)
--- a/plugins/vpn.c
+++ b/plugins/vpn.c
@@ -1438,6 +1438,8 @@ static void destroy_provider(struct connection_data *data)
        if (data->call != NULL)
                dbus_pending_call_cancel(data->call);
 
+       connman_provider_set_data(data->provider, NULL);
+
        connman_provider_put(data->provider);
 
        data->provider = NULL;