Using freed pointer "e"

Reordered code to first to do the comparison and then to release data Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Ander Conselvan de Oliveira <ander.conselvan-de-oliveira@nokia.com> Signed-off-by: Erkki Seppälä <erkki.seppala@vincit.fi> Backported-to-NX-by: Ulrich Sibiller <uli42@gmx.de>

Using freed pointer "e"
8aacb3fa · Erkki Seppälä · Ulrich Sibiller · 1a72bc41 · 8aacb3fa
Commit 8aacb3fa authored Jan 10, 2011 by Erkki Seppälä Committed by Ulrich Sibiller Oct 19, 2016
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

XlibInt.c nx-X11/lib/X11/XlibInt.c +2 -2

No files found.
--- a/nx-X11/lib/X11/XlibInt.c
+++ b/nx-X11/lib/X11/XlibInt.c
@@ -2839,10 +2839,10 @@ _XFreeEventCookies(Display *dpy)
    head = (struct stored_event**)&dpy->cookiejar;
    DL_FOREACH_SAFE(*head, e, tmp) {
-        XFree(e->ev.data);
-        XFree(e);
        if (dpy->cookiejar == e)
            dpy->cookiejar = NULL;
+        XFree(e->ev.data);
+        XFree(e);
    }
 }