backlight: Avoid passing an invalid fd into close()

This commit is contained in:
Rob Bradford
2013-01-10 19:48:56 +00:00
committed by Kristian Høgsberg
parent 409f79a2c7
commit 23d330ec9d
+5 -4
View File
@@ -68,7 +68,8 @@ static long backlight_get(struct backlight *backlight, char *node)
value = strtol(buffer, NULL, 10); value = strtol(buffer, NULL, 10);
ret = value; ret = value;
out: out:
close(fd); if (fd >= 0)
close(fd);
free(path); free(path);
return ret; return ret;
} }
@@ -124,10 +125,10 @@ long backlight_set_brightness(struct backlight *backlight, long brightness)
ret = backlight_get_brightness(backlight); ret = backlight_get_brightness(backlight);
backlight->brightness = ret; backlight->brightness = ret;
out: out:
if (buffer) free(buffer);
free(buffer);
free(path); free(path);
close(fd); if (fd >= 0)
close(fd);
return ret; return ret;
} }