From eff9d66ba6af627543a37ea36567d9ce6d0e28a8 Mon Sep 17 00:00:00 2001 From: Miciah Masters Date: Sat, 24 Dec 2016 03:46:02 -0500 Subject: Do not write NUL to state file Do not write a trailing NUL byte to the state file. Before: % brightnessctl -ms intel_backlight,backlight,1000,20%,4794 % od -t x1z /tmp/brightnessctl/backlight/intel_backlight 0000000 31 30 30 30 00 >1000.< 0000005 % After: % brightnessctl -ms intel_backlight,backlight,1000,20%,4794 % od -t x1z /tmp/brightnessctl/backlight/intel_backlight 0000000 31 30 30 30 >1000< 0000004 % Note that restore still works: % brightnessctl -mr intel_backlight,backlight,1000,20%,4794 % --- brightnessctl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'brightnessctl.c') diff --git a/brightnessctl.c b/brightnessctl.c index 3f7cfc1..1eee6c6 100644 --- a/brightnessctl.c +++ b/brightnessctl.c @@ -405,7 +405,7 @@ int save_device_data(struct device *dev) { goto fail; if (!(fp = fopen(d_path, "w"))) goto fail; - if (fwrite(c, 1, s + 1, fp) < s + 1) + if (fwrite(c, 1, s, fp) < s) errno = -1; fclose(fp); fail: -- cgit v1.2.3