From 7a35b78648a3a0761cbb0263b2d75bb5a6f2a3a2 Mon Sep 17 00:00:00 2001
From: Vadim Yanitskiy <axilirator@gmail.com>
Date: Thu, 21 Nov 2019 21:51:23 +0700
Subject: libosmovty: simplify condition checking vty->fd in vty_close()

On POSIX systems, standard I/O streams - stdin, stdout, and stderr,
always have default file descriptors 0, 1, and 2 respectively.

Change-Id: Ied35d142af0ba0f5ad78975b8f22c35b32d6ff71
---
 src/vty/vty.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/vty/vty.c b/src/vty/vty.c
index 8cc2a3ae..ebdf9fc9 100644
--- a/src/vty/vty.c
+++ b/src/vty/vty.c
@@ -231,8 +231,8 @@ void vty_close(struct vty *vty)
 	/* Unset vector. */
 	vector_unset(vtyvec, vty->fd);
 
-	/* Close socket. */
-	if (vty->fd > 0 && vty->fd != fileno(stderr)) {
+	/* Close socket (ignore standard I/O streams). */
+	if (vty->fd > 2) {
 		close(vty->fd);
 		vty->fd = -1;
 	}
-- 
cgit v1.2.3