Merge pull request #189 from opensourcerouting/config-write-sync

lib: use fsync() for config writes, plug fd leak

1 files changed, 8 insertions, 1 deletions

diff --git a/lib/vty.c b/lib/vty.c
index 2660ca3251..0568351989 100644
--- a/lib/vty.c
+++ b/lib/vty.c
@@ -2254,9 +2254,16 @@ vty_close (struct vty *vty)
   /* Unset vector. */
   vector_unset (vtyvec, vty->fd);
 
+  if (vty->wfd > 0 && vty->type == VTY_FILE)
+    fsync (vty->wfd);
+
   /* Close socket. */
   if (vty->fd > 0)
-    close (vty->fd);
+    {
+      close (vty->fd);
+      if (vty->wfd > 0 && vty->wfd != vty->fd)
+        close (vty->wfd);
+    }
   else
     was_stdio = true;