[PATCH libXrandr 1/2] fix: doGetScreenResources() info: redundant null check on calling free()

walter harms wharms at bfs.de
Thu Jul 28 17:31:10 UTC 2016


janitorial patch: remove some unneeded if() before free()


---
 src/XrrScreen.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/XrrScreen.c b/src/XrrScreen.c
index f29071c..b8ce7e5 100644
--- a/src/XrrScreen.c
+++ b/src/XrrScreen.c
@@ -127,8 +127,8 @@ doGetScreenResources (Display *dpy, Window window, int poll)
     xrsr = (XRRScreenResources *) Xmalloc(rbytes);
     wire_names = (char *) Xmalloc (rep.nbytesNames);
     if (xrsr == NULL || wire_names == NULL) {
-	if (xrsr) Xfree (xrsr);
-	if (wire_names) Xfree (wire_names);
+	Xfree (xrsr);
+	Xfree (wire_names);
 	_XEatDataWords (dpy, rep.length);
 	UnlockDisplay (dpy);
 	SyncHandle ();
-- 
2.1.4



More information about the xorg-devel mailing list