diff options
| author | walter harms <wharms@bfs.de> | 2016-07-28 19:32:46 +0200 |
|---|---|---|
| committer | Hans de Goede <hdegoede@redhat.com> | 2016-08-15 08:56:13 +0200 |
| commit | 8ac94020b018105240ea45a87df2603d1eb5808b (patch) | |
| tree | 815280ea671d85476aa40e3a237f77804dffc9f1 | |
| parent | 4ed36e386b21c1a65d614d5bf2b2c82d1e74ae2e (diff) | |
fix: redundant null check on calling free()
janitorial patch: remove some unneeded if() before free()
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
| -rw-r--r-- | src/XrrMonitor.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/XrrMonitor.c b/src/XrrMonitor.c index 71d3943..a9eaa7b 100644 --- a/src/XrrMonitor.c +++ b/src/XrrMonitor.c @@ -84,8 +84,8 @@ XRRGetMonitors(Display *dpy, Window window, Bool get_active, int *nmonitors) mon = Xmalloc (rbytes); if (buf == NULL || mon == NULL) { - if (buf != NULL) Xfree(buf); - if (mon != NULL) Xfree(mon); + Xfree(buf); + Xfree(mon); _XEatDataWords (dpy, rep.length); UnlockDisplay (dpy); SyncHandle (); @@ -194,7 +194,6 @@ XRRAllocateMonitor(Display *dpy, int noutput) void XRRFreeMonitors(XRRMonitorInfo *monitors) { - if (monitors) - Xfree(monitors); + Xfree(monitors); } |