[chrony-dev] [PATCH] rtc_linux.c: Remove useless assignment `error = -1;`

[ Thread Index | Date Index | More chrony.tuxfamily.org/chrony-dev Archives ]


Date: Thu, 3 Oct 2013 14:37:31 +0200

The Clang static analyzer scan-build from Debian clang version 3.4-1
found the following unneeded assignment.

        rtc_linux.c:756:5: warning: Value stored to 'error' is never read
            error = 1;
            ^       ~

Indeed, if in that if branch, the function returns without ever looking
at the variable `error`. So remove the line.
---
 rtc_linux.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/rtc_linux.c b/rtc_linux.c
index 9fe0ffb..6e1771b 100644
--- a/rtc_linux.c
+++ b/rtc_linux.c
@@ -753,7 +753,6 @@ read_from_device(void *any)
     /* This looks like a bad error : the file descriptor was indicating it was
      * ready to read but we couldn't read anything.  Give up. */
     LOG(LOGS_ERR, LOGF_RtcLinux, "Could not read flags %s : %s", CNF_GetRtcDevice(), strerror(errno));
-    error = 1;
     SCH_RemoveInputFileHandler(fd);
     switch_interrupts(0); /* Likely to raise error too, but just to be sure... */
     close(fd);
-- 
1.8.4.rc3

Attachment: signature.asc
Description: This is a digitally signed message part



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/