]> git.armaanb.net Git - mmenu.git/commitdiff
Patch leaks
authorArmaan Bhojwani <me@armaanb.net>
Sun, 9 May 2021 20:03:18 +0000 (16:03 -0400)
committerArmaan Bhojwani <me@armaanb.net>
Sun, 9 May 2021 20:03:18 +0000 (16:03 -0400)
mmenu.c

diff --git a/mmenu.c b/mmenu.c
index 08f3d06fcea1aa01050c13a8ad510327578fb37c..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();
@@ -116,8 +118,8 @@ getmenu(char *date)
                        strncat(nl, next, 2);
                }
        }
-       free(outp);
 
+       free(outp);
        return nl;
 }
 
@@ -128,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
@@ -165,7 +170,8 @@ main(int argc, char *argv[])
 
        // Create text widget to display menu
        n = 0;
-       XtSetArg(args[n], XmNvalue, getmenu("")); n++;
+       char *nl = getmenu("");
+       XtSetArg(args[n], XmNvalue, nl); n++;
        XtSetArg(args[n], XmNeditable, False); n++;
        XtSetArg(args[n], XmNcolumns, 80); n++;
        XtSetArg(args[n], XmNrows, 20); n++;
@@ -173,6 +179,7 @@ main(int argc, char *argv[])
        XtSetArg(args[n], XmNcursorPositionVisible, False); n++;
        disp = XmCreateScrolledText(rowcol, "text", args, n);
        XtManageChild(disp);
+       free(nl);
 
        // Create day selector
        strs = ArgvToXmStringTable (XtNumber(days), days);