On success, fgets always terminates the result.

If fgets succeeds, then the resulting char array is always
terminated by a '\0'. No need to keep extra space, therefore
sizeof(buf) is the correct argument.

Signed-off-by: Tobias Stoeckmann <[email protected]>

diff --git a/slstatus.c b/slstatus.c
index 6deb68c..ada6441 100644 (file)
--- a/slstatus.c
+++ b/slstatus.c
@@ -447,7 +447,7 @@ run_command(const char *cmd)
                warn("Failed to get command output for %s", cmd);
                return smprintf("%s", UNKNOWN_STR);
        }
                warn("Failed to get command output for %s", cmd);
                return smprintf("%s", UNKNOWN_STR);
        }

-       fgets(buf, sizeof(buf) - 1, fp);
+       fgets(buf, sizeof(buf), fp);

        pclose(fp);
        buf[sizeof(buf) - 1] = '\0';
 
        pclose(fp);
        buf[sizeof(buf) - 1] = '\0';