add braces to single statements so drkhsh doesn't be upset

[slstatus.git] / slstatus.c

diff --git a/slstatus.c b/slstatus.c
index 56f2649..ac4c962 100644 (file)
--- a/slstatus.c
+++ b/slstatus.c
@@ -69,9 +69,8 @@ static char *vol_perc(const char *card);
 static char *wifi_perc(const char *iface);
 static char *wifi_essid(const char *iface);
 static char *kernel_release(void);
-static void set_status(const char *str);
 static void sighandler(const int signo);
-static void usage(void);
+static void usage(int);
 
 char *argv0;
 char concat[];
@@ -700,11 +699,11 @@ wifi_perc(const char *iface)
        fgets(buf, sizeof(buf), fp);
        fclose(fp);
 
-       datastart = strstr(buf, concat);
-       if (datastart != NULL) {
-               datastart = strstr(buf, ":");
-               sscanf(datastart + 1, " %*d   %d  %*d  %*d                %*d      %*d          %*d              %*d      %*d            %*d", &perc);
+       if ((datastart = strstr(buf, concat)) == NULL) {
+               return smprintf("%s", UNKNOWN_STR);
        }
+       datastart = (datastart+(strlen(iface)+1));
+       sscanf(datastart + 1, " %*d   %d  %*d  %*d                %*d      %*d          %*d              %*d      %*d            %*d", &perc);
 
        return smprintf("%d%%", perc);
 }
@@ -718,7 +717,8 @@ wifi_essid(const char *iface)
 
        memset(&wreq, 0, sizeof(struct iwreq));
        wreq.u.essid.length = IW_ESSID_MAX_SIZE+1;
-       sprintf(wreq.ifr_name, iface);
+       snprintf(wreq.ifr_name, sizeof(wreq.ifr_name), "%s", iface);
+
        if (sockfd == -1) {
                warn("Failed to get ESSID for interface %s", iface);
                return smprintf("%s", UNKNOWN_STR);
@@ -747,13 +747,6 @@ kernel_release(void)
        return smprintf("%s", udata.release);
 }
 
-static void
-set_status(const char *str)
-{
-       XStoreName(dpy, DefaultRootWindow(dpy), str);
-       XSync(dpy, False);
-}
-
 static void
 sighandler(const int signo)
 {
@@ -763,10 +756,10 @@ sighandler(const int signo)
 }
 
 static void
-usage(void)
+usage(int eval)
 {
-       fprintf(stderr, "usage: %s [-dhov]\n", argv0);
-       exit(1);
+       fprintf(stderr, "usage: %s [-d] [-o] [-v] [-h]\n", argv0);
+       exit(eval);
 }
 
 int
@@ -788,12 +781,14 @@ main(int argc, char *argv[])
                case 'v':
                        printf("slstatus %s (C) 2016 slstatus engineers\n", VERSION);
                        return 0;
+               case 'h':
+                       usage(0);
                default:
-                       usage();
+                       usage(1);
        } ARGEND
 
        if (dflag && oflag) {
-               usage();
+               usage(1);
        }
        if (dflag && daemon(1, 1) < 0) {
                err(1, "daemon");
@@ -825,22 +820,20 @@ main(int argc, char *argv[])
                                element = smprintf("%s", UNKNOWN_STR);
                                warnx("Failed to format output");
                        }
-                       if (strlcat(status_string, element, sizeof(status_string)) >= sizeof(status_string))
+                       if (strlcat(status_string, element, sizeof(status_string)) >= sizeof(status_string)) {
                                warnx("Output too long");
+                       }
                        free(res);
                        free(element);
                }
 
                if (!oflag) {
-                       set_status(status_string);
+                       XStoreName(dpy, DefaultRootWindow(dpy), status_string);
+                       XSync(dpy, False);
                } else {
                        printf("%s\n", status_string);
                }
 
-               /*
-                * subtract delay time spend in function
-                * calls from the actual global delay time
-                */
                if ((UPDATE_INTERVAL - delay) <= 0) {
                        delay = 0;
                        continue;
@@ -851,7 +844,7 @@ main(int argc, char *argv[])
        }
 
        if (!oflag) {
-               set_status(NULL);
+               XStoreName(dpy, DefaultRootWindow(dpy), NULL);
                XCloseDisplay(dpy);
        }