diff options
author | Aaron Marcher <info@nulltime.net> | 2016-09-14 16:47:28 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-09-14 16:47:28 +0200 |
commit | ad03218fd54e01d612b19a0a569f440131cd5800 (patch) | |
tree | 2d121b922e034f20c9ee274306f0030bd09afe92 /slstatus.c | |
parent | 5609b9d9dec24b8771280e9eace955fe17663236 (diff) | |
parent | 561de544e74487f5e7a9b51850d382fe2278a915 (diff) |
Merge pull request #18 from R41z/master
simplified run_command()
Diffstat (limited to 'slstatus.c')
-rw-r--r-- | slstatus.c | 16 |
1 files changed, 4 insertions, 12 deletions
@@ -406,26 +406,18 @@ ram_used(void) static char * run_command(const char* command) { - int good; FILE *fp = popen(command, "r"); - char buffer[64] = ""; + char buffer[64] = '\0'; if (fp == NULL) { warn("Could not get command output for: %s", command); return smprintf(UNKNOWN_STR); } - fgets(buffer, sizeof(buffer)-1, fp); - pclose(fp); - for (int i = 0 ; i != sizeof(buffer); i++) { - if (buffer[i] == '\0') { - good = 1; - break; - } - } - if (good) - buffer[strlen(buffer)-1] = '\0'; + fgets(buffer, sizeof(buffer), fp); + buffer[sizeof(buffer)-1] = '\0'; + pclose(fp); return smprintf("%s", buffer); } |