From: Daniel Wagner <[email protected]>

After we have created and written all options and configuration
for for the VPN daemon, we should also close the file handles.

Reported by coverity.
---
 vpn/plugins/l2tp.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/vpn/plugins/l2tp.c b/vpn/plugins/l2tp.c
index d60ebbe..6651c7f 100644
--- a/vpn/plugins/l2tp.c
+++ b/vpn/plugins/l2tp.c
@@ -671,6 +671,8 @@ static int run_connect(struct vpn_provider *provider,
 
        g_free(l2tp_name);
        g_free(pppd_name);
+       close(l2tp_fd);
+       close(pppd_fd);
 
        err = connman_task_run(task, l2tp_died, provider,
                                NULL, NULL, NULL);
-- 
1.8.4.474.g128a96c

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

Reply via email to