helpers.c:set_hostname() - Fix resource and memory leaks.

Signed-off-by: Joachim Nilsson <troglobit@gmail.com>

finit.c

@@ -250,7 +250,7 @@ int main(int argc, char* argv[])
 	run("/sbin/sysctl -e -p /etc/sysctl.conf >/dev/null");
 
 	/* Set initial hostname. */
-	set_hostname(hostname);
+	set_hostname(&hostname);
 
 	ifconfig("lo", "127.0.0.1", "255.0.0.0", 1);
 	if (network)

helpers.c

@@ -580,7 +580,7 @@ pid_t run_getty(char *cmd, char *args[], int console)
 			}
 
 			if (console) {
-				i = write(STDERR_FILENO, msg, sizeof(msg));
+				(void)write(STDERR_FILENO, msg, sizeof(msg));
 				while (read(STDIN_FILENO, &c, 1) == 1 && c != '\n')
 					continue;
 			}
@@ -697,26 +697,29 @@ void do_sleep(unsigned int sec)
 		;
 }
 
-void set_hostname(char *hostname)
+void set_hostname(char **hostname)
 {
 	FILE *fp;
 
-	_d("Set hostname: %s", hostname);
+	_d("Set hostname: %s", *hostname);
-	if ((fp = fopen("/etc/hostname", "r")) != NULL) {
+	fp = fopen("/etc/hostname", "r");
+	if (fp) {
 		struct stat st;
 
 		fstat(fileno(fp), &st);
-		hostname = realloc(hostname, st.st_size);
+		*hostname = realloc(*hostname, st.st_size);
-		if (!hostname)
+		if (!*hostname) {
+			fclose(fp);
 			return;
+		}
 
-		fgets(hostname, st.st_size, fp);
+		fgets(*hostname, st.st_size, fp);
-		chomp(hostname);
+		chomp(*hostname);
 		fclose(fp);
 	}
 
-	if (hostname)
+	if (*hostname)
-		sethostname(hostname, strlen(hostname));
+		sethostname(*hostname, strlen(*hostname));
 }
 
 /**

helpers.h

@@ -101,7 +101,7 @@ void    chomp           (char *str);
 void    do_sleep        (unsigned int sec);
 int     getuser         (char *username);
 int     getgroup        (char *group);
-void    set_hostname    (char *hostname);
+void    set_hostname    (char **hostname);
 
 int     complete        (char *cmd, int pid);
 int     run             (char *cmd);