From: Juergen Gross <jgross@suse.com>
Date: Wed, 22 Feb 2017 15:28:45 +0000 (+0100)
Subject: xenstore: correct test for opened logfile in reopen_log()
X-Git-Tag: archive/raspbian/4.11.1-1+rpi1~1^2~66^2~2715
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=d7ee5c3c7ad9a23441efc2a7553aa1df6996959d;p=xen.git

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 <jgross@suse.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
---

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);