From: Sami Kerola <kerolasa@iki.fi>
Date: Sun, 23 Jul 2017 21:20:37 +0000 (+0100)
Subject: wdctl: do not call fsync(2) when closing watchdog device
X-Git-Tag: v2.31-rc1~169
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=4aaabaa9f4a9ed61afb0b9d16f01e0aaf526e8e0;p=thirdparty%2Futil-linux.git

wdctl: do not call fsync(2) when closing watchdog device

This removes bogus write failed warning.

$ wdctl
wdctl: write failed: Invalid argument
Device:        /dev/watchdog
[...]

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
---

diff --git a/sys-utils/wdctl.c b/sys-utils/wdctl.c
index 33012dca36..642db85d22 100644
--- a/sys-utils/wdctl.c
+++ b/sys-utils/wdctl.c
@@ -343,7 +343,7 @@ static int set_watchdog(struct wdinfo *wd, int timeout)
 		warn(_("cannot set timeout for %s"), wd->device);
 	}
 
-	if (close_fd(fd))
+	if (close(fd))
 		warn(_("write failed"));
 	sigprocmask(SIG_SETMASK, &oldsigs, NULL);
 	printf(P_("Timeout has been set to %d second.\n",
@@ -408,7 +408,7 @@ static int read_watchdog(struct wdinfo *wd)
 		 * the machine might end up rebooting. */
 	}
 
-	if (close_fd(fd))
+	if (close(fd))
 		warn(_("write failed"));
 	sigprocmask(SIG_SETMASK, &oldsigs, NULL);