Fix some memory leaks

[slstatus.git] / slstatus.c

diff --git a/slstatus.c b/slstatus.c
index 86d27eb..1db9af8 100644 (file)
--- a/slstatus.c
+++ b/slstatus.c
@@ -3,65 +3,38 @@
 /* global libraries */
 #include <alsa/asoundlib.h>
 #include <fcntl.h>
 /* global libraries */
 #include <alsa/asoundlib.h>
 #include <fcntl.h>

+#include <locale.h>

 #include <stdarg.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <stdarg.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <sys/types.h>
 #include <sys/stat.h>

+#include <sys/statvfs.h>

 #include <time.h>
 #include <unistd.h>
 #include <X11/Xlib.h>
 
 #include <time.h>
 #include <unistd.h>
 #include <X11/Xlib.h>
 

-/* local libraries */
+/* local headers */
+#include "slstatus.h"

 #include "config.h"
 
 #include "config.h"
 

-/* check file macro */
-#define CHECK_FILE(X,Y) do { \
-    if (stat(X,&Y) < 0) return -1; \
-    if (!S_ISREG(Y.st_mode)) return -1; \
-} while (0);
-
-/* functions */
-int config_check();
-void setstatus(char *str);
-char *smprintf(char *fmt, ...);
-char *get_battery();
-char *get_cpu_temperature();
-char *get_cpu_usage();
-char *get_datetime();
-char *get_ram_usage();
-char *get_volume();
-char *get_wifi_signal();
-
-/* global variables */
-static Display *dpy;
-
-/* check configured paths */
-int
-config_check()
-{
-    struct stat fs;
-    CHECK_FILE(batterynowfile, fs);
-    CHECK_FILE(batteryfullfile, fs);
-    CHECK_FILE(tempfile, fs);
-    return 0;
-}
-
-/* set statusbar (WM_NAME) */
+/* set statusbar */

 void
 void

-setstatus(char *str)
+setstatus(const char *str)

 {
 {

+    /* set WM_NAME via X11 */

     XStoreName(dpy, DefaultRootWindow(dpy), str);
     XSync(dpy, False);
 }
 
 /* smprintf function */
 char *
     XStoreName(dpy, DefaultRootWindow(dpy), str);
     XSync(dpy, False);
 }
 
 /* smprintf function */
 char *

-smprintf(char *fmt, ...)
+smprintf(const char *fmt, ...)

 {
     va_list fmtargs;
     char *ret = NULL;
 {
     va_list fmtargs;
     char *ret = NULL;

+

     va_start(fmtargs, fmt);
     if (vasprintf(&ret, fmt, fmtargs) < 0)
         return NULL;
     va_start(fmtargs, fmt);
     if (vasprintf(&ret, fmt, fmtargs) < 0)
         return NULL;


@@ -72,14 +45,28 @@ smprintf(char *fmt, ...)
 
 /* battery percentage */
 char *
 
 /* battery percentage */
 char *

-get_battery()
+get_battery(const char *battery)

 {
     int now, full, perc;
 {
     int now, full, perc;

+    char batterynowfile[64] = "";
+    char batteryfullfile[64] = "";

     FILE *fp;
 
     FILE *fp;
 

+    /* generate battery nowfile path */
+    strcat(batterynowfile, batterypath);
+    strcat(batterynowfile, battery);
+    strcat(batterynowfile, "/");
+    strcat(batterynowfile, batterynow);
+
+    /* generate battery fullfile path */
+    strcat(batteryfullfile, batterypath);
+    strcat(batteryfullfile, battery);
+    strcat(batteryfullfile, "/");
+    strcat(batteryfullfile, batteryfull);
+

     /* open battery now file */
     if (!(fp = fopen(batterynowfile, "r"))) {
     /* open battery now file */
     if (!(fp = fopen(batterynowfile, "r"))) {

-        fprintf(stderr, "Error opening battery file.");
+        fprintf(stderr, "Error opening battery file.%s",batterynowfile);

         return smprintf("n/a");
     }
 
         return smprintf("n/a");
     }
 


@@ -110,13 +97,13 @@ get_battery()
 
 /* cpu temperature */
 char *
 
 /* cpu temperature */
 char *

-get_cpu_temperature()
+get_cpu_temperature(const char *file)

 {
     int temperature;
     FILE *fp;
 
     /* open temperature file */
 {
     int temperature;
     FILE *fp;
 
     /* open temperature file */

-    if (!(fp = fopen(tempfile, "r"))) {
+    if (!(fp = fopen(file, "r"))) {

         fprintf(stderr, "Could not open temperature file.\n");
         return smprintf("n/a");
     }
         fprintf(stderr, "Could not open temperature file.\n");
         return smprintf("n/a");
     }


@@ -133,7 +120,7 @@ get_cpu_temperature()
 
 /* cpu percentage */
 char *
 
 /* cpu percentage */
 char *

-get_cpu_usage()
+get_cpu_usage(const char *null)

 {
     int perc;
     long double a[4], b[4];
 {
     int perc;
     long double a[4], b[4];


@@ -175,7 +162,7 @@ get_cpu_usage()
 
 /* date and time */
 char *
 
 /* date and time */
 char *

-get_datetime()
+get_datetime(const char *timeformat)

 {
     time_t tm;
     size_t bufsize = 64;
 {
     time_t tm;
     size_t bufsize = 64;


@@ -183,18 +170,43 @@ get_datetime()
 
     /* get time in format */
     time(&tm);
 
     /* get time in format */
     time(&tm);

+    setlocale(LC_TIME, "");

     if(!strftime(buf, bufsize, timeformat, localtime(&tm))) {
     if(!strftime(buf, bufsize, timeformat, localtime(&tm))) {

+        setlocale(LC_TIME, "C");

         fprintf(stderr, "Strftime failed.\n");
         return smprintf("n/a");
     }
 
         fprintf(stderr, "Strftime failed.\n");
         return smprintf("n/a");
     }
 

+    setlocale(LC_TIME, "C");

     /* return time */
     /* return time */

-    return smprintf("%s", buf);
+    char *ret = smprintf("%s", buf);
+    free(buf);
+    return ret;
+}
+
+/* disk usage percentage */
+char *
+get_diskusage(const char *mountpoint)
+{
+    int perc = 0;
+    struct statvfs fs;
+
+    /* try to open mountpoint */
+    if (statvfs(mountpoint, &fs) < 0) {
+        fprintf(stderr, "Could not get filesystem info.\n");
+        return smprintf("n/a");
+    }
+
+    /* calculate percent */
+    perc = 100 * (1.0f - ((float)fs.f_bavail / (float)fs.f_blocks));
+
+    /* return perc */
+    return smprintf("%d%%", perc);

 }
 
 /* ram percentage */
 char *
 }
 
 /* ram percentage */
 char *

-get_ram_usage()
+get_ram_usage(const char *null)

 {
     int perc;
     long total, free, buffers, cached;
 {
     int perc;
     long total, free, buffers, cached;


@@ -224,7 +236,7 @@ get_ram_usage()
 
 /* alsa volume percentage */
 char *
 
 /* alsa volume percentage */
 char *

-get_volume()
+get_volume(const char *soundcard)

 {
     int mute = 0;
     long vol = 0, max = 0, min = 0;
 {
     int mute = 0;
     long vol = 0, max = 0, min = 0;


@@ -262,24 +274,22 @@ get_volume()
 
 /* wifi percentage */
 char *
 
 /* wifi percentage */
 char *

-get_wifi_signal()
+get_wifi_signal(const char *wificard)

 {
     int bufsize = 255;
     int strength;
     char buf[bufsize];
     char *datastart;
 {
     int bufsize = 255;
     int strength;
     char buf[bufsize];
     char *datastart;

-    char path_start[16] = "/sys/class/net/";
-    char path_end[11] = "/operstate";
-    char path[32];
+    char path[64];

     char status[5];
     char needle[sizeof wificard + 1];
     FILE *fp;
 
     /* generate the path name */
     memset(path, 0, sizeof path);
     char status[5];
     char needle[sizeof wificard + 1];
     FILE *fp;
 
     /* generate the path name */
     memset(path, 0, sizeof path);

-    strcat(path, path_start);
+    strcat(path, "/sys/class/net/");

     strcat(path, wificard);
     strcat(path, wificard);

-    strcat(path, path_end);
+    strcat(path, "/operstate");

 
     /* open wifi file */
     if(!(fp = fopen(path, "r"))) {
 
     /* open wifi file */
     if(!(fp = fopen(path, "r"))) {


@@ -326,50 +336,32 @@ get_wifi_signal()
 int
 main()
 {
 int
 main()
 {

-    char status[1024];
-    char *battery = NULL;
-    char *cpu_temperature = NULL;
-    char *cpu_usage = NULL;
-    char *datetime = NULL;
-    char *ram_usage = NULL;
-    char *volume = NULL;
-    char *wifi_signal = NULL;
-
-    /* check config for sanity */
-    if (config_check() < 0) {
-        fprintf(stderr, "Config error, please check paths and recompile\n");
-        exit(1);
-    }
+    char status_string[1024];
+    struct arg argument;

 
 

-    /* open display */
+    /* try to open display */

     if (!(dpy = XOpenDisplay(0x0))) {
         fprintf(stderr, "Cannot open display!\n");
         exit(1);
     }
 
     if (!(dpy = XOpenDisplay(0x0))) {
         fprintf(stderr, "Cannot open display!\n");
         exit(1);
     }
 

-    /* return status every second */
+    /* return status every interval */

     for (;;) {
     for (;;) {

-        /* assign the values */
-        battery = get_battery();
-        cpu_temperature = get_cpu_temperature();
-        cpu_usage = get_cpu_usage();
-        datetime = get_datetime();
-        ram_usage = get_ram_usage();
-        volume = get_volume();
-        wifi_signal = get_wifi_signal();
-
-        /* return the status */
-        sprintf(status, FORMATSTRING, ARGUMENTS);
-        setstatus(status);
-
-        /* free the values */
-        free(battery);
-        free(cpu_temperature);
-        free(cpu_usage);
-        free(datetime);
-        free(ram_usage);
-        free(volume);
-        free(wifi_signal);
+        /* clear the string */
+        memset(status_string, 0, sizeof(status_string));
+
+        /* generate status_string */
+        for (size_t i = 0; i < sizeof(args) / sizeof(args[0]); ++i) {
+            argument = args[i];
+            char *res = argument.func(argument.args);
+            char *element = smprintf(argument.format, res);
+            strcat(status_string, element);
+            free(res);
+            free(element);
+        }
+
+        /* return the statusbar */
+        setstatus(status_string);

 
         /* wait, "update_interval - 1" because of get_cpu_usage() which uses 1 second */
         sleep(update_interval -1);
 
         /* wait, "update_interval - 1" because of get_cpu_usage() which uses 1 second */
         sleep(update_interval -1);