diff options
| author | Tobias Stoeckmann <tobias@stoeckmann.org> | 2017-04-04 21:27:31 +0200 |
|---|---|---|
| committer | Tobias Stoeckmann <tobias@stoeckmann.org> | 2017-04-04 21:27:31 +0200 |
| commit | 35295f190207d9001fdcf30fe76f11434691b259 (patch) | |
| tree | 59c7b103bb8200dda5fade062de850929714aabc | |
| parent | 1289bdb742adab134cf4874447b66ab386deb724 (diff) | |
| download | slstatus-35295f190207d9001fdcf30fe76f11434691b259.tar.gz | |
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 <tobias@stoeckmann.org>
| -rw-r--r-- | slstatus.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -447,7 +447,7 @@ run_command(const char *cmd) 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'; |
