suckless/slstatus

wifi_perc: Fix file descriptor leak (699f6734d99ac67f9f1dceae2b7aeee8cb105d2f)
Repositories | README | LICENSE
commit 699f6734d99ac67f9f1dceae2b7aeee8cb105d2f
parent af3cdfbbc8dfbdb723fdfef8ac08a89fdee25f2e
Author: Aaron Marcher <[email protected]>
Date:   Wed,  2 May 2018 19:41:53 +0200

wifi_perc: Fix file descriptor leak

Diffstat:

Mcomponents/wifi.c5+++--


1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/components/wifi.c b/components/wifi.c
@@ -30,10 +30,11 @@

 			        strerror(errno));
 			return NULL;
 		}
-		if(!(p = fgets(status, 5, fp)) || strcmp(status, "up\n") != 0) {
+		p = fgets(status, 5, fp);
+		fclose(fp);
+		if(!p || strcmp(status, "up\n") != 0) {
 			return NULL;
 		}
-		fclose(fp);
 
 		if (!(fp = fopen("/proc/net/wireless", "r"))) {
 			fprintf(stderr, "fopen '/proc/net/wireless': %s\n",