From b8913bc047474aa0a3bfbf03ecbb3b5050605ed1 Mon Sep 17 00:00:00 2001 From: Ruben Kerkhof Date: Fri, 4 Dec 2015 21:56:40 +0100 Subject: [PATCH] multimeter plugin: fix fd leak If open returns 0 we leak fd. CID #38016 Signed-off-by: Florian Forster --- src/multimeter.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/multimeter.c b/src/multimeter.c index 775eb57e..03608e6c 100644 --- a/src/multimeter.c +++ b/src/multimeter.c @@ -153,9 +153,9 @@ static int multimeter_init (void) for (i = 0; i < 10; i++) { - device[strlen(device)-1] = i + '0'; + device[strlen(device)-1] = i + '0'; - if ((fd = open(device, O_RDWR | O_NOCTTY)) > 0) + if ((fd = open(device, O_RDWR | O_NOCTTY)) != -1) { struct termios tios; int rts = TIOCM_RTS; @@ -171,7 +171,7 @@ static int multimeter_init (void) tcflush(fd, TCIFLUSH); tcsetattr(fd, TCSANOW, &tios); ioctl(fd, TIOCMBIC, &rts); - + if (multimeter_read_value (&value) < -1) { close (fd); -- 2.30.2