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.30.2~24
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=0b526bec22c86b2151d7fea955874b6e58b5a647;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 e47fc70e98..483189de49 100644
--- a/sys-utils/wdctl.c
+++ b/sys-utils/wdctl.c
@@ -344,7 +344,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",
@@ -409,7 +409,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);