From: Nikolaus Rath <Nikolaus@rath.org>
Date: Mon, 5 Jun 2017 11:27:31 +0000 (-0400)
Subject: getgroups(): clarify code
X-Git-Tag: fuse-3.1.0~14
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=60a0f6b00ea069e1eb0d9adb1c0234c9d2dd63f1;p=qemu-gpiodev%2Flibfuse.git

getgroups(): clarify code

read() return value should always be positive or -1. However,
since we cast to unsigned a little later, it's clearer
to check for non-negativity.
---

diff --git a/lib/fuse_lowlevel.c b/lib/fuse_lowlevel.c
index c3724a6..c1c573f 100644
--- a/lib/fuse_lowlevel.c
+++ b/lib/fuse_lowlevel.c
@@ -2935,7 +2935,7 @@ retry:
 
 	ret = read(fd, buf, bufsize);
 	close(fd);
-	if (ret == -1) {
+	if (ret < 0) {
 		ret = -EIO;
 		goto out_free;
 	}