From: Brian Behlendorf <behlendorf1@llnl.gov>
Date: Fri, 14 Aug 2009 17:11:03 +0000 (-0700)
Subject: All calls to ftruncate() must have their return code checked.
X-Git-Tag: zfs-0.4.6~50^2^2^2^2^2^2^2^2^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e6b541db64b6bd103807f9215e2d2356954d28df;p=zfs

All calls to ftruncate() must have their return code checked.
Under FC11 rpm builds by default add the --fortify-source option which
ensures that functions flagged with certain attributes must have their
return codes checked.  Normally this is just a warning but we always
build with -Werror so this is fatal.  Simply wrap the function in a
verify call to ensure we catch a failure if there is one.
---

diff --git a/cmd/ztest/ztest.c b/cmd/ztest/ztest.c
index 746db0c07..0e9d079a9 100644
--- a/cmd/ztest/ztest.c
+++ b/cmd/ztest/ztest.c
@@ -1161,7 +1161,7 @@ grow_vdev(vdev_t *vd, void *arg)
 		return (vd);
 
 	fsize = lseek(fd, 0, SEEK_END);
-	(void) ftruncate(fd, *newsize);
+	VERIFY(ftruncate(fd, *newsize) == 0);
 
 	if (zopt_verbose >= 6) {
 		(void) printf("%s grew from %lu to %lu bytes\n",