summaryrefslogtreecommitdiff
path: root/slstatus.c
diff options
context:
space:
mode:
authorAaron Marcher <info@nulltime.net>2016-09-14 16:47:28 +0200
committerGitHub <noreply@github.com>2016-09-14 16:47:28 +0200
commitad03218fd54e01d612b19a0a569f440131cd5800 (patch)
tree2d121b922e034f20c9ee274306f0030bd09afe92 /slstatus.c
parent5609b9d9dec24b8771280e9eace955fe17663236 (diff)
parent561de544e74487f5e7a9b51850d382fe2278a915 (diff)
Merge pull request #18 from R41z/master
simplified run_command()
Diffstat (limited to 'slstatus.c')
-rw-r--r--slstatus.c16
1 files changed, 4 insertions, 12 deletions
diff --git a/slstatus.c b/slstatus.c
index fefdf6f..a13bf3b 100644
--- a/slstatus.c
+++ b/slstatus.c
@@ -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);
}