From d7ee5c3c7ad9a23441efc2a7553aa1df6996959d Mon Sep 17 00:00:00 2001 From: Juergen Gross Date: Wed, 22 Feb 2017 16:28:45 +0100 Subject: [PATCH] xenstore: correct test for opened logfile in reopen_log() As 0 is a valid file descriptor testing a descriptor to be valid should be done via >= 0 instead of > 0. Signed-off-by: Juergen Gross Acked-by: Wei Liu --- tools/xenstore/xenstored_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/xenstore/xenstored_core.c b/tools/xenstore/xenstored_core.c index 1e9b62280b..7b16338d82 100644 --- a/tools/xenstore/xenstored_core.c +++ b/tools/xenstore/xenstored_core.c @@ -209,7 +209,7 @@ static void trigger_reopen_log(int signal __attribute__((unused))) static void reopen_log(void) { if (tracefile) { - if (tracefd > 0) + if (tracefd >= 0) close(tracefd); tracefd = open(tracefile, O_WRONLY|O_CREAT|O_APPEND, 0600); -- 2.30.2