Patch leaks

[mmenu.git] / mmenu.c

diff --git a/mmenu.c b/mmenu.c
index 03d7bced2b24d65615c2412cfa802493c872692b..4e4283ca7bb5c555bc212303aede7819ec07ad05 100644 (file)
--- a/mmenu.c
+++ b/mmenu.c
@@ -101,6 +101,8 @@ getmenu(char *date)
                if (c == '>') intag = false;
        }
 
+       free(chunk.memory);
+
        // Strip empty newlines
        char *nl = (char *) calloc(1, sizeof(char));
        if (!nl) memfail();
@@ -108,12 +110,15 @@ getmenu(char *date)
        j = 1;
        for (int i = 0; outp[i]; i++) {
                j++;
-               if (outp[i] == '\n' && outp[i+1] == '\n') i+=3;
-               nl = (char *) realloc(nl, j);
-               if (!nl) memfail();
-               next[0] = outp[i+1];
-               strncat(nl, next, 2);
+               if (!((outp[i] == '\n' && outp[i+1] == '\n') ||
+                               (outp[i] == ' ' && outp[i+1] == ' '))) {
+                       nl = (char *) realloc(nl, j);
+                       if (!nl) memfail();
+                       next[0] = outp[i+1];
+                       strncat(nl, next, 2);
+               }
        }
+
        free(outp);
        return nl;
 }
@@ -125,7 +130,10 @@ dayback(Widget unused, XtPointer client_data, XtPointer call_data)
        Widget disp = (Widget) client_data;
        XmListCallbackStruct *list_cbs = (XmListCallbackStruct *) call_data;
        char *nl = getmenu(days[list_cbs->item_position - 1]);
-       XmTextReplace((Widget) disp, 0, strlen(XmTextGetString(disp)), nl);
+       char *oldtext = XmTextGetString(disp) ;
+       XmTextReplace((Widget) disp, 0, strlen(oldtext), nl);
+       free(nl);
+       free(oldtext);
 }
 
 // Convert an array of string to an array of compound strings
@@ -158,6 +166,7 @@ main(int argc, char *argv[])
        int n = 0;
        XtSetArg (args[n], XmNorientation, XmHORIZONTAL); n++;
        rowcol = XmCreateRowColumn(toplevel, "rowcol", args, n);
+       XtManageChild(rowcol);
 
        // Create text widget to display menu
        n = 0;
@@ -165,9 +174,12 @@ main(int argc, char *argv[])
        XtSetArg(args[n], XmNvalue, nl); n++;
        XtSetArg(args[n], XmNeditable, False); n++;
        XtSetArg(args[n], XmNcolumns, 80); n++;
+       XtSetArg(args[n], XmNrows, 20); n++;
+       XtSetArg (args[n], XmNeditMode, XmMULTI_LINE_EDIT); n++;
        XtSetArg(args[n], XmNcursorPositionVisible, False); n++;
        disp = XmCreateScrolledText(rowcol, "text", args, n);
        XtManageChild(disp);
+       free(nl);
 
        // Create day selector
        strs = ArgvToXmStringTable (XtNumber(days), days);
@@ -175,7 +187,6 @@ main(int argc, char *argv[])
        XtVaSetValues(daysel, XmNitems, strs, XmNitemCount, XtNumber(days), NULL);
        XtAddCallback(daysel, XmNbrowseSelectionCallback, dayback, (XtPointer) disp);
        XtManageChild(daysel);
-       XtManageChild(rowcol);
 
        // Display everything
        XtRealizeWidget(toplevel);