From 0b0be8eaf77366af5a108c7ec0d40c9a501231ea Mon Sep 17 00:00:00 2001 From: Rob Bradford Date: Wed, 5 Dec 2012 18:47:06 +0000 Subject: [PATCH] backlight: Remove unnecessary and confusing NULL checks At this point path must point to an allocated string since otherwise the asprintf that makes the allocation would have failed and we would have returned earlier. --- src/libbacklight.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/libbacklight.c b/src/libbacklight.c index df811b41..2ac135a5 100644 --- a/src/libbacklight.c +++ b/src/libbacklight.c @@ -69,8 +69,7 @@ static long backlight_get(struct backlight *backlight, char *node) ret = value; out: close(fd); - if (path) - free(path); + free(path); return ret; } @@ -127,8 +126,7 @@ long backlight_set_brightness(struct backlight *backlight, long brightness) out: if (buffer) free(buffer); - if (path) - free(path); + free(path); close(fd); return ret; }