From: Quentin Rameau <quinq+hackers@fifth.space>
Date: Wed, 18 Nov 2015 21:45:26 +0000 (+0100)
Subject: drw.c: Avoid potential memory leak in drw_cur_create()
X-Git-Url: https://git.brendanfh.com/?a=commitdiff_plain;h=996a4c5c6e7b98c9f3ef50846f567b5d6a2da932;p=sent.git

drw.c: Avoid potential memory leak in drw_cur_create()

If drw was NULL, memory was still allocated for cur.
---

diff --git a/drw.c b/drw.c
index a2dfe6c..15797ce 100644
--- a/drw.c
+++ b/drw.c
@@ -396,9 +396,9 @@ drw_font_getexts(Fnt *font, const char *text, unsigned int len, unsigned int *w,
 
 Cur *
 drw_cur_create(Drw *drw, int shape) {
-	Cur *cur = (Cur *)calloc(1, sizeof(Cur));
+	Cur *cur;
 
-	if(!drw || !cur)
+	if(!drw || !(cur = (Cur *)calloc(1, sizeof(Cur))))
 		return NULL;
 	cur->cursor = XCreateFontCursor(drw->dpy, shape);
 	return cur;