From: Jo-Philipp Wich <jow@openwrt.org>
Date: Thu, 3 May 2012 17:19:22 +0000 (+0000)
Subject: uhttpd: display errors in init script, code formatting changes, bump package version
X-Git-Tag: v19.07.0-rc1~24713
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=8e9d914343967982a1a48e80000bbb09161e636c;p=openwrt%2Fstaging%2Fhauke.git

uhttpd: display errors in init script, code formatting changes, bump package version

SVN-Revision: 31572
---

diff --git a/package/uhttpd/Makefile b/package/uhttpd/Makefile
index 5688073add..0331470bf3 100644
--- a/package/uhttpd/Makefile
+++ b/package/uhttpd/Makefile
@@ -8,7 +8,7 @@
 include $(TOPDIR)/rules.mk
 
 PKG_NAME:=uhttpd
-PKG_RELEASE:=31
+PKG_RELEASE:=32
 
 PKG_BUILD_DIR := $(BUILD_DIR)/$(PKG_NAME)
 PKG_CONFIG_DEPENDS := \
diff --git a/package/uhttpd/files/uhttpd.init b/package/uhttpd/files/uhttpd.init
index c370f90109..d4037a1b98 100755
--- a/package/uhttpd/files/uhttpd.init
+++ b/package/uhttpd/files/uhttpd.init
@@ -108,6 +108,11 @@ start_instance()
 
 	SERVICE_PID_FILE=/var/run/uhttpd_${cfg}.pid
 	service_start $UHTTPD_BIN -f $UHTTPD_ARGS
+
+	# Check if daemon is running, if not then
+	# re-execute in foreground to display error.
+	sleep 1 && service_check $UHTTPD_BIN || \
+		$UHTTPD_BIN -f $UHTTPD_ARGS
 }
 
 stop_instance()
diff --git a/package/uhttpd/src/uhttpd-cgi.c b/package/uhttpd/src/uhttpd-cgi.c
index 16bfda7dba..f852125698 100644
--- a/package/uhttpd/src/uhttpd-cgi.c
+++ b/package/uhttpd/src/uhttpd-cgi.c
@@ -30,9 +30,9 @@ static struct http_response * uh_cgi_header_parse(char *buf, int len, int *off)
 	static struct http_response res;
 
 
-	if( ((bufptr = strfind(buf, len, "\r\n\r\n", 4)) != NULL) ||
-	    ((bufptr = strfind(buf, len, "\n\n", 2)) != NULL)
-	) {
+	if (((bufptr = strfind(buf, len, "\r\n\r\n", 4)) != NULL) ||
+	    ((bufptr = strfind(buf, len, "\n\n", 2)) != NULL))
+	{
 		*off = (int)(bufptr - buf) + ((bufptr[0] == '\r') ? 4 : 2);
 
 		memset(&res, 0, sizeof(res));
@@ -42,45 +42,48 @@ static struct http_response * uh_cgi_header_parse(char *buf, int len, int *off)
 
 		bufptr = &buf[0];
 
-		for( pos = 0; pos < *off; pos++ )
+		for (pos = 0; pos < *off; pos++)
 		{
-			if( !hdrname && (buf[pos] == ':') )
+			if (!hdrname && (buf[pos] == ':'))
 			{
 				buf[pos++] = 0;
 
-				if( (pos < len) && (buf[pos] == ' ') )
+				if ((pos < len) && (buf[pos] == ' '))
 					pos++;
 
-				if( pos < len )
+				if (pos < len)
 				{
 					hdrname = bufptr;
 					bufptr = &buf[pos];
 				}
 			}
 
-			else if( (buf[pos] == '\r') || (buf[pos] == '\n') )
+			else if ((buf[pos] == '\r') || (buf[pos] == '\n'))
 			{
-				if( ! hdrname )
+				if (! hdrname)
 					break;
 
 				buf[pos++] = 0;
 
-				if( (pos < len) && (buf[pos] == '\n') )
+				if ((pos < len) && (buf[pos] == '\n'))
 					pos++;
 
-				if( pos <= len )
+				if (pos <= len)
 				{
-					if( (hdrcount + 1) < array_size(res.headers) )
+					if ((hdrcount + 1) < array_size(res.headers))
 					{
-						if( ! strcasecmp(hdrname, "Status") )
+						if (!strcasecmp(hdrname, "Status"))
 						{
 							res.statuscode = atoi(bufptr);
 
-							if( res.statuscode < 100 )
+							if (res.statuscode < 100)
 								res.statuscode = 200;
 
-							if( ((bufptr = strchr(bufptr, ' ')) != NULL) && (&bufptr[1] != 0) )
+							if (((bufptr = strchr(bufptr, ' ')) != NULL) &&
+								(&bufptr[1] != 0))
+							{
 								res.statusmsg = &bufptr[1];
+							}
 						}
 						else
 						{
@@ -105,29 +108,30 @@ static struct http_response * uh_cgi_header_parse(char *buf, int len, int *off)
 	return NULL;
 }
 
-static char * uh_cgi_header_lookup(struct http_response *res, const char *hdrname)
+static char * uh_cgi_header_lookup(struct http_response *res,
+								   const char *hdrname)
 {
 	int i;
 
 	foreach_header(i, res->headers)
 	{
-		if( ! strcasecmp(res->headers[i], hdrname) )
+		if (!strcasecmp(res->headers[i], hdrname))
 			return res->headers[i+1];
 	}
 
 	return NULL;
 }
 
-static int uh_cgi_error_500(struct client *cl, struct http_request *req, const char *message)
+static int uh_cgi_error_500(struct client *cl, struct http_request *req,
+							const char *message)
 {
-	if( uh_http_sendf(cl, NULL,
-		"HTTP/%.1f 500 Internal Server Error\r\n"
-		"Content-Type: text/plain\r\n%s\r\n",
-			req->version, 
-			(req->version > 1.0)
-				? "Transfer-Encoding: chunked\r\n" : ""
-		) >= 0
-	) {
+	if (uh_http_sendf(cl, NULL,
+					  "HTTP/%.1f 500 Internal Server Error\r\n"
+					  "Content-Type: text/plain\r\n%s\r\n",
+					  req->version,
+					  (req->version > 1.0)
+					      ? "Transfer-Encoding: chunked\r\n" : "") >= 0)
+	{
 		return uh_http_send(cl, req, message, -1);
 	}
 
@@ -135,10 +139,9 @@ static int uh_cgi_error_500(struct client *cl, struct http_request *req, const c
 }
 
 
-void uh_cgi_request(
-	struct client *cl, struct http_request *req,
-	struct path_info *pi, struct interpreter *ip
-) {
+void uh_cgi_request(struct client *cl, struct http_request *req,
+					struct path_info *pi, struct interpreter *ip)
+{
 	int i, hdroff, bufoff, rv;
 	int hdrlen = 0;
 	int buflen = 0;
@@ -165,26 +168,26 @@ void uh_cgi_request(
 
 
 	/* spawn pipes for me->child, child->me */
-	if( (pipe(rfd) < 0) || (pipe(wfd) < 0) )
+	if ((pipe(rfd) < 0) || (pipe(wfd) < 0))
 	{
 		uh_http_sendhf(cl, 500, "Internal Server Error",
-			"Failed to create pipe: %s", strerror(errno));
+					   "Failed to create pipe: %s", strerror(errno));
 
-		if( rfd[0] > 0 ) close(rfd[0]);
-		if( rfd[1] > 0 ) close(rfd[1]);
-		if( wfd[0] > 0 ) close(wfd[0]);
-		if( wfd[1] > 0 ) close(wfd[1]);
+		if (rfd[0] > 0) close(rfd[0]);
+		if (rfd[1] > 0) close(rfd[1]);
+		if (wfd[0] > 0) close(wfd[0]);
+		if (wfd[1] > 0) close(wfd[1]);
 
 		return;
 	}
 
 	/* fork off child process */
-	switch( (child = fork()) )
+	switch ((child = fork()))
 	{
 		/* oops */
 		case -1:
 			uh_http_sendhf(cl, 500, "Internal Server Error",
-				"Failed to fork child: %s", strerror(errno));
+						   "Failed to fork child: %s", strerror(errno));
 			return;
 
 		/* exec child */
@@ -212,9 +215,9 @@ void uh_cgi_request(
 			fd_cloexec(wfd[0]);
 
 			/* check for regular, world-executable file _or_ interpreter */
-			if( ((pi->stat.st_mode & S_IFREG) &&
-			     (pi->stat.st_mode & S_IXOTH)) || (ip != NULL)
-			) {
+			if (((pi->stat.st_mode & S_IFREG) &&
+			     (pi->stat.st_mode & S_IXOTH)) || (ip != NULL))
+			{
 				/* build environment */
 				clearenv();
 
@@ -225,7 +228,7 @@ void uh_cgi_request(
 
 #ifdef HAVE_TLS
 				/* https? */
-				if( cl->tls )
+				if (cl->tls)
 					setenv("HTTPS", "on", 1);
 #endif
 
@@ -243,11 +246,11 @@ void uh_cgi_request(
 				setenv("DOCUMENT_ROOT", pi->root, 1);
 				setenv("QUERY_STRING", pi->query ? pi->query : "", 1);
 
-				if( pi->info )
+				if (pi->info)
 					setenv("PATH_INFO", pi->info, 1);
 
 				/* REDIRECT_STATUS, php-cgi wants it */
-				switch( req->redirect_status )
+				switch (req->redirect_status)
 				{
 					case 404:
 						setenv("REDIRECT_STATUS", "404", 1);
@@ -259,13 +262,13 @@ void uh_cgi_request(
 				}
 
 				/* http version */
-				if( req->version > 1.0 )
+				if (req->version > 1.0)
 					setenv("SERVER_PROTOCOL", "HTTP/1.1", 1);
 				else
 					setenv("SERVER_PROTOCOL", "HTTP/1.0", 1);
 
 				/* request method */
-				switch( req->method )
+				switch (req->method)
 				{
 					case UH_HTTP_MSG_GET:
 						setenv("REQUEST_METHOD", "GET", 1);
@@ -284,75 +287,71 @@ void uh_cgi_request(
 				setenv("REQUEST_URI", req->url, 1);
 
 				/* remote user */
-				if( req->realm )
+				if (req->realm)
 					setenv("REMOTE_USER", req->realm->user, 1);
 
 				/* request message headers */
 				foreach_header(i, req->headers)
 				{
-					if( ! strcasecmp(req->headers[i], "Accept") )
+					if (!strcasecmp(req->headers[i], "Accept"))
 						setenv("HTTP_ACCEPT", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "Accept-Charset") )
+					else if (!strcasecmp(req->headers[i], "Accept-Charset"))
 						setenv("HTTP_ACCEPT_CHARSET", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "Accept-Encoding") )
+					else if (!strcasecmp(req->headers[i], "Accept-Encoding"))
 						setenv("HTTP_ACCEPT_ENCODING", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "Accept-Language") )
+					else if (!strcasecmp(req->headers[i], "Accept-Language"))
 						setenv("HTTP_ACCEPT_LANGUAGE", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "Authorization") )
+					else if (!strcasecmp(req->headers[i], "Authorization"))
 						setenv("HTTP_AUTHORIZATION", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "Connection") )
+					else if (!strcasecmp(req->headers[i], "Connection"))
 						setenv("HTTP_CONNECTION", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "Cookie") )
+					else if (!strcasecmp(req->headers[i], "Cookie"))
 						setenv("HTTP_COOKIE", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "Host") )
+					else if (!strcasecmp(req->headers[i], "Host"))
 						setenv("HTTP_HOST", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "Referer") )
+					else if (!strcasecmp(req->headers[i], "Referer"))
 						setenv("HTTP_REFERER", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "User-Agent") )
+					else if (!strcasecmp(req->headers[i], "User-Agent"))
 						setenv("HTTP_USER_AGENT", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "Content-Type") )
+					else if (!strcasecmp(req->headers[i], "Content-Type"))
 						setenv("CONTENT_TYPE", req->headers[i+1], 1);
 
-					else if( ! strcasecmp(req->headers[i], "Content-Length") )
+					else if (!strcasecmp(req->headers[i], "Content-Length"))
 						setenv("CONTENT_LENGTH", req->headers[i+1], 1);
 				}
 
 
 				/* execute child code ... */
-				if( chdir(pi->root) )
+				if (chdir(pi->root))
 					perror("chdir()");
 
-				if( ip != NULL )
+				if (ip != NULL)
 					execl(ip->path, ip->path, pi->phys, NULL);
 				else
 					execl(pi->phys, pi->phys, NULL);
 
 				/* in case it fails ... */
-				printf(
-					"Status: 500 Internal Server Error\r\n\r\n"
-					"Unable to launch the requested CGI program:\n"
-					"  %s: %s\n",
-						ip ? ip->path : pi->phys, strerror(errno)
-				);
+				printf("Status: 500 Internal Server Error\r\n\r\n"
+					   "Unable to launch the requested CGI program:\n"
+					   "  %s: %s\n",
+					   ip ? ip->path : pi->phys, strerror(errno));
 			}
 
 			/* 403 */
 			else
 			{
-				printf(
-					"Status: 403 Forbidden\r\n\r\n"
-					"Access to this resource is forbidden\n"
-				);
+				printf("Status: 403 Forbidden\r\n\r\n"
+					   "Access to this resource is forbidden\n");
 			}
 
 			close(wfd[0]);
@@ -371,11 +370,11 @@ void uh_cgi_request(
 			fd_max = max(rfd[0], wfd[1]) + 1;
 
 			/* find content length */
-			if( req->method == UH_HTTP_MSG_POST )
+			if (req->method == UH_HTTP_MSG_POST)
 			{
 				foreach_header(i, req->headers)
 				{
-					if( ! strcasecmp(req->headers[i], "Content-Length") )
+					if (!strcasecmp(req->headers[i], "Content-Length"))
 					{
 						content_length = atoi(req->headers[i+1]);
 						break;
@@ -387,7 +386,7 @@ void uh_cgi_request(
 			memset(hdr, 0, sizeof(hdr));
 
 			/* I/O loop, watch our pipe ends and dispatch child reads/writes from/to socket */
-			while( 1 )
+			while (1)
 			{
 				FD_ZERO(&reader);
 				FD_ZERO(&writer);
@@ -399,31 +398,33 @@ void uh_cgi_request(
 				timeout.tv_usec = 0;
 
 				ensure_out(rv = select_intr(fd_max, &reader,
-					(content_length > -1) ? &writer : NULL, NULL, &timeout));
+											(content_length > -1)
+												? &writer : NULL,
+											NULL, &timeout));
 
 				/* timeout */
-				if( rv == 0 )
+				if (rv == 0)
 				{
 					ensure_out(kill(child, 0));
 				}
 
 				/* wait until we can read or write or both */
-				else if( rv > 0 )
+				else if (rv > 0)
 				{
 					/* ready to write to cgi program */
-					if( FD_ISSET(wfd[1], &writer) )
+					if (FD_ISSET(wfd[1], &writer))
 					{
 						/* there is unread post data waiting */
-						if( content_length > 0 )
+						if (content_length > 0)
 						{
 							/* read it from socket ... */
 							ensure_out(buflen = uh_tcp_recv(cl, buf,
 								min(content_length, sizeof(buf))));
 
-							if( buflen > 0 )
+							if (buflen > 0)
 							{
 								/* ... and write it to child's stdin */
-								if( write(wfd[1], buf, buflen) < 0 )
+								if (write(wfd[1], buf, buflen) < 0)
 									perror("write()");
 
 								content_length -= buflen;
@@ -432,7 +433,7 @@ void uh_cgi_request(
 							/* unexpected eof! */
 							else
 							{
-								if( write(wfd[1], "", 0) < 0 )
+								if (write(wfd[1], "", 0) < 0)
 									perror("write()");
 
 								content_length = 0;
@@ -440,7 +441,7 @@ void uh_cgi_request(
 						}
 
 						/* there is no more post data, close pipe to child's stdin */
-						else if( content_length > -1 )
+						else if (content_length > -1)
 						{
 							close(wfd[1]);
 							content_length = -1;
@@ -448,16 +449,16 @@ void uh_cgi_request(
 					}
 
 					/* ready to read from cgi program */
-					if( FD_ISSET(rfd[0], &reader) )
+					if (FD_ISSET(rfd[0], &reader))
 					{
 						/* read data from child ... */
-						if( (buflen = read(rfd[0], buf, sizeof(buf))) > 0 )
+						if ((buflen = read(rfd[0], buf, sizeof(buf))) > 0)
 						{
 							/* we have not pushed out headers yet, parse input */
-							if( ! header_sent )
+							if (!header_sent)
 							{
 								/* head buffer not full and no end yet */
-								if( hdrlen < sizeof(hdr) )
+								if (hdrlen < sizeof(hdr))
 								{
 									bufoff = min(buflen, sizeof(hdr) - hdrlen);
 									memcpy(&hdr[hdrlen], buf, bufoff);
@@ -470,7 +471,7 @@ void uh_cgi_request(
 
 
 								/* try to parse header ... */
-								if( (res = uh_cgi_header_parse(hdr, hdrlen, &hdroff)) != NULL )
+								if ((res = uh_cgi_header_parse(hdr, hdrlen, &hdroff)) != NULL)
 								{
 									/* write status */
 									ensure_out(uh_http_sendf(cl, NULL,
@@ -506,12 +507,12 @@ void uh_cgi_request(
 									ensure_out(uh_http_send(cl, NULL, "\r\n", -1));
 
 									/* push out remaining head buffer */
-									if( hdroff < hdrlen )
+									if (hdroff < hdrlen)
 										ensure_out(uh_http_send(cl, req, &hdr[hdroff], hdrlen - hdroff));
 								}
 
 								/* ... failed and head buffer exceeded */
-								else if( hdrlen >= sizeof(hdr) )
+								else if (hdrlen >= sizeof(hdr))
 								{
 									ensure_out(uh_cgi_error_500(cl, req,
 										"The CGI program generated an invalid response:\n\n"));
@@ -526,7 +527,7 @@ void uh_cgi_request(
 								}
 
 								/* push out remaining read buffer */
-								if( bufoff < buflen )
+								if (bufoff < buflen)
 									ensure_out(uh_http_send(cl, req, &buf[bufoff], buflen - bufoff));
 
 								header_sent = 1;
@@ -542,7 +543,7 @@ void uh_cgi_request(
 						else
 						{
 							/* cgi script did not output useful stuff at all */
-							if( ! header_sent )
+							if (!header_sent)
 							{
 								/* I would do this ...
 								 *
@@ -576,7 +577,7 @@ void uh_cgi_request(
 				/* timeout exceeded or interrupted by SIGCHLD */
 				else
 				{
-					if( (errno != EINTR) && ! header_sent )
+					if ((errno != EINTR) && ! header_sent)
 					{
 						ensure_out(uh_http_sendhf(cl, 504, "Gateway Timeout",
 							"The CGI script took too long to produce "
@@ -594,7 +595,7 @@ void uh_cgi_request(
 			close(rfd[0]);
 			close(wfd[1]);
 
-			if( !kill(child, 0) )
+			if (!kill(child, 0))
 			{
 				kill(child, SIGTERM);
 				waitpid(child, NULL, 0);
@@ -603,4 +604,3 @@ void uh_cgi_request(
 			break;
 	}
 }
-
diff --git a/package/uhttpd/src/uhttpd-file.c b/package/uhttpd/src/uhttpd-file.c
index 816a69124e..0d9a207b11 100644
--- a/package/uhttpd/src/uhttpd-file.c
+++ b/package/uhttpd/src/uhttpd-file.c
@@ -31,13 +31,13 @@ static const char * uh_file_mime_lookup(const char *path)
 	struct mimetype *m = &uh_mime_types[0];
 	const char *e;
 
-	while( m->extn )
+	while (m->extn)
 	{
 		e = &path[strlen(path)-1];
 
-		while( e >= path )
+		while (e >= path)
 		{
-			if( (*e == '.' || *e == '/') && !strcasecmp(&e[1], m->extn) )
+			if ((*e == '.' || *e == '/') && !strcasecmp(&e[1], m->extn))
 				return m->mime;
 
 			e--;
@@ -54,10 +54,9 @@ static const char * uh_file_mktag(struct stat *s)
 	static char tag[128];
 
 	snprintf(tag, sizeof(tag), "\"%x-%x-%x\"",
-		(unsigned int) s->st_ino,
-		(unsigned int) s->st_size,
-		(unsigned int) s->st_mtime
-	);
+			 (unsigned int) s->st_ino,
+			 (unsigned int) s->st_size,
+			 (unsigned int) s->st_mtime);
 
 	return tag;
 }
@@ -68,7 +67,7 @@ static time_t uh_file_date2unix(const char *date)
 
 	memset(&t, 0, sizeof(t));
 
-	if( strptime(date, "%a, %d %b %Y %H:%M:%S %Z", &t) != NULL )
+	if (strptime(date, "%a, %d %b %Y %H:%M:%S %Z", &t) != NULL)
 		return timegm(&t);
 
 	return 0;
@@ -90,7 +89,7 @@ static char * uh_file_header_lookup(struct http_request *req, const char *name)
 
 	foreach_header(i, req->headers)
 	{
-		if( ! strcasecmp(req->headers[i], name) )
+		if (!strcasecmp(req->headers[i], name))
 			return req->headers[i+1];
 	}
 
@@ -98,26 +97,30 @@ static char * uh_file_header_lookup(struct http_request *req, const char *name)
 }
 
 
-static int uh_file_response_ok_hdrs(struct client *cl, struct http_request *req, struct stat *s)
+static int uh_file_response_ok_hdrs(struct client *cl, struct http_request *req,
+									struct stat *s)
 {
 	ensure_ret(uh_http_sendf(cl, NULL, "Connection: close\r\n"));
 
-	if( s )
+	if (s)
 	{
 		ensure_ret(uh_http_sendf(cl, NULL, "ETag: %s\r\n", uh_file_mktag(s)));
-		ensure_ret(uh_http_sendf(cl, NULL, "Last-Modified: %s\r\n", uh_file_unix2date(s->st_mtime)));
+		ensure_ret(uh_http_sendf(cl, NULL, "Last-Modified: %s\r\n",
+								 uh_file_unix2date(s->st_mtime)));
 	}
 
 	return uh_http_sendf(cl, NULL, "Date: %s\r\n", uh_file_unix2date(time(NULL)));
 }
 
-static int uh_file_response_200(struct client *cl, struct http_request *req, struct stat *s)
+static int uh_file_response_200(struct client *cl, struct http_request *req,
+								struct stat *s)
 {
 	ensure_ret(uh_http_sendf(cl, NULL, "HTTP/%.1f 200 OK\r\n", req->version));
 	return uh_file_response_ok_hdrs(cl, req, s);
 }
 
-static int uh_file_response_304(struct client *cl, struct http_request *req, struct stat *s)
+static int uh_file_response_304(struct client *cl, struct http_request *req,
+								struct stat *s)
 {
 	ensure_ret(uh_http_sendf(cl, NULL, "HTTP/%.1f 304 Not Modified\r\n", req->version));
 	return uh_file_response_ok_hdrs(cl, req, s);
@@ -130,25 +133,26 @@ static int uh_file_response_412(struct client *cl, struct http_request *req)
 		"Connection: close\r\n", req->version);
 }
 
-static int uh_file_if_match(struct client *cl, struct http_request *req, struct stat *s, int *ok)
+static int uh_file_if_match(struct client *cl, struct http_request *req,
+							struct stat *s, int *ok)
 {
 	const char *tag = uh_file_mktag(s);
 	char *hdr = uh_file_header_lookup(req, "If-Match");
 	char *p;
 	int i;
 
-	if( hdr )
+	if (hdr)
 	{
 		p = &hdr[0];
 
-		for( i = 0; i < strlen(hdr); i++ )
+		for (i = 0; i < strlen(hdr); i++)
 		{
-			if( (hdr[i] == ' ') || (hdr[i] == ',') )
+			if ((hdr[i] == ' ') || (hdr[i] == ','))
 			{
 				hdr[i++] = 0;
 				p = &hdr[i];
 			}
-			else if( !strcmp(p, "*") || !strcmp(p, tag) )
+			else if (!strcmp(p, "*") || !strcmp(p, tag))
 			{
 				*ok = 1;
 				return *ok;
@@ -164,14 +168,16 @@ static int uh_file_if_match(struct client *cl, struct http_request *req, struct
 	return *ok;
 }
 
-static int uh_file_if_modified_since(struct client *cl, struct http_request *req, struct stat *s, int *ok)
+static int uh_file_if_modified_since(struct client *cl,
+									 struct http_request *req, struct stat *s,
+									 int *ok)
 {
 	char *hdr = uh_file_header_lookup(req, "If-Modified-Since");
 	*ok = 1;
 
-	if( hdr )
+	if (hdr)
 	{
-		if( uh_file_date2unix(hdr) >= s->st_mtime )
+		if (uh_file_date2unix(hdr) >= s->st_mtime)
 		{
 			*ok = 0;
 			ensure_ret(uh_file_response_304(cl, req, s));
@@ -181,7 +187,8 @@ static int uh_file_if_modified_since(struct client *cl, struct http_request *req
 	return *ok;
 }
 
-static int uh_file_if_none_match(struct client *cl, struct http_request *req, struct stat *s, int *ok)
+static int uh_file_if_none_match(struct client *cl, struct http_request *req,
+								 struct stat *s, int *ok)
 {
 	const char *tag = uh_file_mktag(s);
 	char *hdr = uh_file_header_lookup(req, "If-None-Match");
@@ -189,26 +196,30 @@ static int uh_file_if_none_match(struct client *cl, struct http_request *req, st
 	int i;
 	*ok = 1;
 
-	if( hdr )
+	if (hdr)
 	{
 		p = &hdr[0];
 
-		for( i = 0; i < strlen(hdr); i++ )
+		for (i = 0; i < strlen(hdr); i++)
 		{
-			if( (hdr[i] == ' ') || (hdr[i] == ',') )
+			if ((hdr[i] == ' ') || (hdr[i] == ','))
 			{
 				hdr[i++] = 0;
 				p = &hdr[i];
 			}
-			else if( !strcmp(p, "*") || !strcmp(p, tag) )
+			else if (!strcmp(p, "*") || !strcmp(p, tag))
 			{
 				*ok = 0;
 
-				if( (req->method == UH_HTTP_MSG_GET) ||
-				    (req->method == UH_HTTP_MSG_HEAD) )
+				if ((req->method == UH_HTTP_MSG_GET) ||
+				    (req->method == UH_HTTP_MSG_HEAD))
+				{
 					ensure_ret(uh_file_response_304(cl, req, s));
+				}
 				else
+				{
 					ensure_ret(uh_file_response_412(cl, req));
+				}
 
 				break;
 			}
@@ -218,12 +229,13 @@ static int uh_file_if_none_match(struct client *cl, struct http_request *req, st
 	return *ok;
 }
 
-static int uh_file_if_range(struct client *cl, struct http_request *req, struct stat *s, int *ok)
+static int uh_file_if_range(struct client *cl, struct http_request *req,
+							struct stat *s, int *ok)
 {
 	char *hdr = uh_file_header_lookup(req, "If-Range");
 	*ok = 1;
 
-	if( hdr )
+	if (hdr)
 	{
 		*ok = 0;
 		ensure_ret(uh_file_response_412(cl, req));
@@ -232,14 +244,16 @@ static int uh_file_if_range(struct client *cl, struct http_request *req, struct
 	return *ok;
 }
 
-static int uh_file_if_unmodified_since(struct client *cl, struct http_request *req, struct stat *s, int *ok)
+static int uh_file_if_unmodified_since(struct client *cl,
+									   struct http_request *req, struct stat *s,
+									   int *ok)
 {
 	char *hdr = uh_file_header_lookup(req, "If-Unmodified-Since");
 	*ok = 1;
 
-	if( hdr )
+	if (hdr)
 	{
-		if( uh_file_date2unix(hdr) <= s->st_mtime )
+		if (uh_file_date2unix(hdr) <= s->st_mtime)
 		{
 			*ok = 0;
 			ensure_ret(uh_file_response_412(cl, req));
@@ -255,7 +269,8 @@ static int uh_file_scandir_filter_dir(const struct dirent *e)
 	return strcmp(e->d_name, ".") ? 1 : 0;
 }
 
-static void uh_file_dirlist(struct client *cl, struct http_request *req, struct path_info *pi)
+static void uh_file_dirlist(struct client *cl, struct http_request *req,
+							struct path_info *pi)
 {
 	int i;
 	int count = 0;
@@ -265,54 +280,60 @@ static void uh_file_dirlist(struct client *cl, struct http_request *req, struct
 	struct stat s;
 
 	ensure_out(uh_http_sendf(cl, req,
-		"<html><head><title>Index of %s</title></head>"
-		"<body><h1>Index of %s</h1><hr /><ol>",
-			pi->name, pi->name
-	));
+							 "<html><head><title>Index of %s</title></head>"
+							 "<body><h1>Index of %s</h1><hr /><ol>",
+							 pi->name, pi->name));
 
-	if( (count = scandir(pi->phys, &files, uh_file_scandir_filter_dir, alphasort)) > 0 )
+	if ((count = scandir(pi->phys, &files, uh_file_scandir_filter_dir,
+						 alphasort)) > 0)
 	{
 		memset(filename, 0, sizeof(filename));
 		memcpy(filename, pi->phys, sizeof(filename));
 		pathptr = &filename[strlen(filename)];
 
 		/* list subdirs */
-		for( i = 0; i < count; i++ )
+		for (i = 0; i < count; i++)
 		{
 			strncat(filename, files[i]->d_name,
-				sizeof(filename) - strlen(files[i]->d_name));
+					sizeof(filename) - strlen(files[i]->d_name));
 
-			if( !stat(filename, &s) &&
-			    (s.st_mode & S_IFDIR) && (s.st_mode & S_IXOTH)
-			)
+			if (!stat(filename, &s) &&
+				(s.st_mode & S_IFDIR) && (s.st_mode & S_IXOTH))
+			{
 				ensure_out(uh_http_sendf(cl, req,
-					"<li><strong><a href='%s%s'>%s</a>/</strong><br />"
-					"<small>modified: %s<br />directory - %.02f kbyte"
-					"<br /><br /></small></li>",
-						pi->name, files[i]->d_name, files[i]->d_name,
-						uh_file_unix2date(s.st_mtime), s.st_size / 1024.0
-				));
+										 "<li><strong><a href='%s%s'>%s</a>/"
+										 "</strong><br /><small>modified: %s"
+										 "<br />directory - %.02f kbyte<br />"
+										 "<br /></small></li>",
+										 pi->name, files[i]->d_name,
+										 files[i]->d_name,
+										 uh_file_unix2date(s.st_mtime),
+										 s.st_size / 1024.0));
+			}
 
 			*pathptr = 0;
 		}
 
 		/* list files */
-		for( i = 0; i < count; i++ )
+		for (i = 0; i < count; i++)
 		{
 			strncat(filename, files[i]->d_name,
-				sizeof(filename) - strlen(files[i]->d_name));
+					sizeof(filename) - strlen(files[i]->d_name));
 
-			if( !stat(filename, &s) &&
-			    !(s.st_mode & S_IFDIR) && (s.st_mode & S_IROTH)
-			)
+			if (!stat(filename, &s) &&
+				!(s.st_mode & S_IFDIR) && (s.st_mode & S_IROTH))
+			{
 				ensure_out(uh_http_sendf(cl, req,
-					"<li><strong><a href='%s%s'>%s</a></strong><br />"
-					"<small>modified: %s<br />%s - %.02f kbyte<br />"
-					"<br /></small></li>",
-						pi->name, files[i]->d_name, files[i]->d_name,
-						uh_file_unix2date(s.st_mtime),
-						uh_file_mime_lookup(filename), s.st_size / 1024.0
-				));
+										 "<li><strong><a href='%s%s'>%s</a>"
+										 "</strong><br /><small>modified: %s"
+										 "<br />%s - %.02f kbyte<br />"
+										 "<br /></small></li>",
+										 pi->name, files[i]->d_name,
+										 files[i]->d_name,
+										 uh_file_unix2date(s.st_mtime),
+										 uh_file_mime_lookup(filename),
+										 s.st_size / 1024.0));
+			}
 
 			*pathptr = 0;
 		}
@@ -322,9 +343,9 @@ static void uh_file_dirlist(struct client *cl, struct http_request *req, struct
 	ensure_out(uh_http_sendf(cl, req, ""));
 
 out:
-	if( files )
+	if (files)
 	{
-		for( i = 0; i < count; i++ )
+		for (i = 0; i < count; i++)
 			free(files[i]);
 
 		free(files);
@@ -340,16 +361,16 @@ void uh_file_request(struct client *cl, struct http_request *req, struct path_in
 	char buf[UH_LIMIT_MSGHEAD];
 
 	/* we have a file */
-	if( (pi->stat.st_mode & S_IFREG) && ((fd = open(pi->phys, O_RDONLY)) > 0) )
+	if ((pi->stat.st_mode & S_IFREG) && ((fd = open(pi->phys, O_RDONLY)) > 0))
 	{
 		/* test preconditions */
-		if(ok) ensure_out(uh_file_if_modified_since(cl, req, &pi->stat, &ok));
-		if(ok) ensure_out(uh_file_if_match(cl, req, &pi->stat, &ok));
-		if(ok) ensure_out(uh_file_if_range(cl, req, &pi->stat, &ok));
-		if(ok) ensure_out(uh_file_if_unmodified_since(cl, req, &pi->stat, &ok));
-		if(ok) ensure_out(uh_file_if_none_match(cl, req, &pi->stat, &ok));
+		if (ok) ensure_out(uh_file_if_modified_since(cl, req, &pi->stat, &ok));
+		if (ok) ensure_out(uh_file_if_match(cl, req, &pi->stat, &ok));
+		if (ok) ensure_out(uh_file_if_range(cl, req, &pi->stat, &ok));
+		if (ok) ensure_out(uh_file_if_unmodified_since(cl, req, &pi->stat, &ok));
+		if (ok) ensure_out(uh_file_if_none_match(cl, req, &pi->stat, &ok));
 
-		if( ok > 0 )
+		if (ok > 0)
 		{
 			/* write status */
 			ensure_out(uh_file_response_200(cl, req, &pi->stat));
@@ -358,17 +379,17 @@ void uh_file_request(struct client *cl, struct http_request *req, struct path_in
 			ensure_out(uh_http_sendf(cl, NULL, "Content-Length: %i\r\n", pi->stat.st_size));
 
 			/* if request was HTTP 1.1 we'll respond chunked */
-			if( (req->version > 1.0) && (req->method != UH_HTTP_MSG_HEAD) )
+			if ((req->version > 1.0) && (req->method != UH_HTTP_MSG_HEAD))
 				ensure_out(uh_http_send(cl, NULL, "Transfer-Encoding: chunked\r\n", -1));
 
 			/* close header */
 			ensure_out(uh_http_send(cl, NULL, "\r\n", -1));
 
 			/* send body */
-			if( req->method != UH_HTTP_MSG_HEAD )
+			if (req->method != UH_HTTP_MSG_HEAD)
 			{
 				/* pump file data */
-				while( (rlen = read(fd, buf, sizeof(buf))) > 0 )
+				while ((rlen = read(fd, buf, sizeof(buf))) > 0)
 					ensure_out(uh_http_send(cl, req, buf, rlen));
 
 				/* send trailer in chunked mode */
@@ -384,12 +405,12 @@ void uh_file_request(struct client *cl, struct http_request *req, struct path_in
 	}
 
 	/* directory */
-	else if( (pi->stat.st_mode & S_IFDIR) && !cl->server->conf->no_dirlists )
+	else if ((pi->stat.st_mode & S_IFDIR) && !cl->server->conf->no_dirlists)
 	{
 		/* write status */
 		ensure_out(uh_file_response_200(cl, req, NULL));
 
-		if( req->version > 1.0 )
+		if (req->version > 1.0)
 			ensure_out(uh_http_send(cl, NULL, "Transfer-Encoding: chunked\r\n", -1));
 
 		ensure_out(uh_http_send(cl, NULL, "Content-Type: text/html\r\n\r\n", -1));
@@ -406,7 +427,6 @@ void uh_file_request(struct client *cl, struct http_request *req, struct path_in
 	}
 
 out:
-	if( fd > -1 )
+	if (fd > -1)
 		close(fd);
 }
-
diff --git a/package/uhttpd/src/uhttpd-lua.c b/package/uhttpd/src/uhttpd-lua.c
index 83b0b0a26b..ea6f26cc9b 100644
--- a/package/uhttpd/src/uhttpd-lua.c
+++ b/package/uhttpd/src/uhttpd-lua.c
@@ -31,7 +31,7 @@ static int uh_lua_recv(lua_State *L)
 
 	length = luaL_checknumber(L, 1);
 
-	if( (length > 0) && (length <= sizeof(buffer)) )
+	if ((length > 0) && (length <= sizeof(buffer)))
 	{
 		FD_ZERO(&reader);
 		FD_SET(fileno(stdin), &reader);
@@ -41,13 +41,13 @@ static int uh_lua_recv(lua_State *L)
 		timeout.tv_usec = 100000;
 
 		/* check whether fd is readable */
-		if( select(fileno(stdin) + 1, &reader, NULL, NULL, &timeout) > 0 )
+		if (select(fileno(stdin) + 1, &reader, NULL, NULL, &timeout) > 0)
 		{
 			/* receive data */
 			rlen = read(fileno(stdin), buffer, length);
 			lua_pushnumber(L, rlen);
 
-			if( rlen > 0 )
+			if (rlen > 0)
 			{
 				lua_pushlstring(L, buffer, rlen);
 				return 2;
@@ -75,9 +75,9 @@ static int uh_lua_send_common(lua_State *L, int chunked)
 
 	buffer = luaL_checklstring(L, 1, &length);
 
-	if( chunked )
+	if (chunked)
 	{
-		if( length > 0 )
+		if (length > 0)
 		{
 			snprintf(chunk, sizeof(chunk), "%X\r\n", length);
 			slen =  write(fileno(stdout), chunk, strlen(chunk));
@@ -117,7 +117,7 @@ static int uh_lua_str2str(lua_State *L, int (*xlate_func) (char *, int, const ch
 
 	inbuf = luaL_checklstring(L, 1, &inlen);
 	outlen = (* xlate_func)(outbuf, sizeof(outbuf), inbuf, inlen);
-	if( outlen < 0 )
+	if (outlen < 0)
 		luaL_error( L, "%s on URL-encode codec",
 			(outlen==-1) ? "buffer overflow" : "malformed string" );
 
@@ -177,7 +177,7 @@ lua_State * uh_lua_init(const struct config *conf)
 
 
 	/* load Lua handler */
-	switch( luaL_loadfile(L, conf->lua_handler) )
+	switch (luaL_loadfile(L, conf->lua_handler))
 	{
 		case LUA_ERRSYNTAX:
 			fprintf(stderr,
@@ -196,7 +196,7 @@ lua_State * uh_lua_init(const struct config *conf)
 
 		default:
 			/* compile Lua handler */
-			switch( lua_pcall(L, 0, 0, 0) )
+			switch (lua_pcall(L, 0, 0, 0))
 			{
 				case LUA_ERRRUN:
 					err_str = luaL_checkstring(L, -1);
@@ -218,7 +218,7 @@ lua_State * uh_lua_init(const struct config *conf)
 					/* test handler function */
 					lua_getglobal(L, UH_LUA_CALLBACK);
 
-					if( ! lua_isfunction(L, -1) )
+					if (! lua_isfunction(L, -1))
 					{
 						fprintf(stderr,
 							"Lua handler provides no " UH_LUA_CALLBACK "(), unable to continue\n");
@@ -260,21 +260,21 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 
 
 	/* spawn pipes for me->child, child->me */
-	if( (pipe(rfd) < 0) || (pipe(wfd) < 0) )
+	if ((pipe(rfd) < 0) || (pipe(wfd) < 0))
 	{
 		uh_http_sendhf(cl, 500, "Internal Server Error",
 			"Failed to create pipe: %s", strerror(errno));
 
-		if( rfd[0] > 0 ) close(rfd[0]);
-		if( rfd[1] > 0 ) close(rfd[1]);
-		if( wfd[0] > 0 ) close(wfd[0]);
-		if( wfd[1] > 0 ) close(wfd[1]);
+		if (rfd[0] > 0) close(rfd[0]);
+		if (rfd[1] > 0) close(rfd[1]);
+		if (wfd[0] > 0) close(wfd[0]);
+		if (wfd[1] > 0) close(wfd[1]);
 
 		return;
 	}
 
 
-	switch( (child = fork()) )
+	switch ((child = fork()))
 	{
 		case -1:
 			uh_http_sendhf(cl, 500, "Internal Server Error",
@@ -329,12 +329,12 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 			lua_setfield(L, -2, "SCRIPT_NAME");
 
 			/* query string, path info */
-			if( (query_string = strchr(req->url, '?')) != NULL )
+			if ((query_string = strchr(req->url, '?')) != NULL)
 			{
 				lua_pushstring(L, query_string + 1);
 				lua_setfield(L, -2, "QUERY_STRING");
 
-				if( (int)(query_string - req->url) > strlen(prefix) )
+				if ((int)(query_string - req->url) > strlen(prefix))
 				{
 					lua_pushlstring(L,
 						&req->url[strlen(prefix)],
@@ -344,7 +344,7 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 					lua_setfield(L, -2, "PATH_INFO");
 				}
 			}
-			else if( strlen(req->url) > strlen(prefix) )
+			else if (strlen(req->url) > strlen(prefix))
 			{
 				lua_pushstring(L, &req->url[strlen(prefix)]);
 				lua_setfield(L, -2, "PATH_INFO");
@@ -354,7 +354,7 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 			lua_pushnumber(L, floor(req->version * 10) / 10);
 			lua_setfield(L, -2, "HTTP_VERSION");
 
-			if( req->version > 1.0 )
+			if (req->version > 1.0)
 				lua_pushstring(L, "HTTP/1.1");
 			else
 				lua_pushstring(L, "HTTP/1.0");
@@ -378,12 +378,12 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 			/* essential env vars */
 			foreach_header(i, req->headers)
 			{
-				if( !strcasecmp(req->headers[i], "Content-Length") )
+				if (!strcasecmp(req->headers[i], "Content-Length"))
 				{
 					lua_pushnumber(L, atoi(req->headers[i+1]));
 					lua_setfield(L, -2, "CONTENT_LENGTH");
 				}
-				else if( !strcasecmp(req->headers[i], "Content-Type") )
+				else if (!strcasecmp(req->headers[i], "Content-Type"))
 				{
 					lua_pushstring(L, req->headers[i+1]);
 					lua_setfield(L, -2, "CONTENT_TYPE");
@@ -407,13 +407,13 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 
 
 			/* call */
-			switch( lua_pcall(L, 1, 0, 0) )
+			switch (lua_pcall(L, 1, 0, 0))
 			{
 				case LUA_ERRMEM:
 				case LUA_ERRRUN:
 					err_str = luaL_checkstring(L, -1);
 
-					if( ! err_str )
+					if (! err_str)
 						err_str = "Unknown error";
 
 					printf(
@@ -447,11 +447,11 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 			fd_max = max(rfd[0], wfd[1]) + 1;
 
 			/* find content length */
-			if( req->method == UH_HTTP_MSG_POST )
+			if (req->method == UH_HTTP_MSG_POST)
 			{
 				foreach_header(i, req->headers)
 				{
-					if( ! strcasecmp(req->headers[i], "Content-Length") )
+					if (! strcasecmp(req->headers[i], "Content-Length"))
 					{
 						content_length = atoi(req->headers[i+1]);
 						break;
@@ -461,7 +461,7 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 
 
 #define ensure(x) \
-	do { if( x < 0 ) goto out; } while(0)
+	do { if (x < 0) goto out; } while(0)
 
 			data_sent = 0;
 
@@ -469,7 +469,7 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 			timeout.tv_usec = 0;
 
 			/* I/O loop, watch our pipe ends and dispatch child reads/writes from/to socket */
-			while( 1 )
+			while (1)
 			{
 				FD_ZERO(&reader);
 				FD_ZERO(&writer);
@@ -478,21 +478,22 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 				FD_SET(wfd[1], &writer);
 
 				/* wait until we can read or write or both */
-				if( select_intr(fd_max, &reader,
-				    (content_length > -1) ? &writer : NULL, NULL,
-					(data_sent < 1) ? &timeout : NULL) > 0
-				) {
+				if (select_intr(fd_max, &reader,
+								(content_length > -1) ? &writer : NULL,
+								NULL,
+								(data_sent < 1) ? &timeout : NULL) > 0)
+				{
 					/* ready to write to Lua child */
-					if( FD_ISSET(wfd[1], &writer) )
+					if (FD_ISSET(wfd[1], &writer))
 					{
 						/* there is unread post data waiting */
-						if( content_length > 0 )
+						if (content_length > 0)
 						{
 							/* read it from socket ... */
-							if( (buflen = uh_tcp_recv(cl, buf, min(content_length, sizeof(buf)))) > 0 )
+							if ((buflen = uh_tcp_recv(cl, buf, min(content_length, sizeof(buf)))) > 0)
 							{
 								/* ... and write it to child's stdin */
-								if( write(wfd[1], buf, buflen) < 0 )
+								if (write(wfd[1], buf, buflen) < 0)
 									perror("write()");
 
 								content_length -= buflen;
@@ -501,7 +502,7 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 							/* unexpected eof! */
 							else
 							{
-								if( write(wfd[1], "", 0) < 0 )
+								if (write(wfd[1], "", 0) < 0)
 									perror("write()");
 
 								content_length = 0;
@@ -509,7 +510,7 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 						}
 
 						/* there is no more post data, close pipe to child's stdin */
-						else if( content_length > -1 )
+						else if (content_length > -1)
 						{
 							close(wfd[1]);
 							content_length = -1;
@@ -517,10 +518,10 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 					}
 
 					/* ready to read from Lua child */
-					if( FD_ISSET(rfd[0], &reader) )
+					if (FD_ISSET(rfd[0], &reader))
 					{
 						/* read data from child ... */
-						if( (buflen = read(rfd[0], buf, sizeof(buf))) > 0 )
+						if ((buflen = read(rfd[0], buf, sizeof(buf))) > 0)
 						{
 							/* pass through buffer to socket */
 							ensure(uh_tcp_send(cl, buf, buflen));
@@ -531,7 +532,7 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 						else
 						{
 							/* error? */
-							if( ! data_sent )
+							if (!data_sent)
 								uh_http_sendhf(cl, 500, "Internal Server Error",
 									"The Lua child did not produce any response");
 
@@ -543,7 +544,7 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 				/* timeout exceeded or interrupted by SIGCHLD */
 				else
 				{
-					if( (errno != EINTR) && ! data_sent )
+					if ((errno != EINTR) && ! data_sent)
 					{
 						ensure(uh_http_sendhf(cl, 504, "Gateway Timeout",
 							"The Lua script took too long to produce "
@@ -558,7 +559,7 @@ void uh_lua_request(struct client *cl, struct http_request *req, lua_State *L)
 			close(rfd[0]);
 			close(wfd[1]);
 
-			if( !kill(child, 0) )
+			if (!kill(child, 0))
 			{
 				kill(child, SIGTERM);
 				waitpid(child, NULL, 0);
diff --git a/package/uhttpd/src/uhttpd-utils.c b/package/uhttpd/src/uhttpd-utils.c
index 1dac33db0f..18969e74dd 100644
--- a/package/uhttpd/src/uhttpd-utils.c
+++ b/package/uhttpd/src/uhttpd-utils.c
@@ -41,7 +41,7 @@ const char * sa_straddr(void *sa)
 	struct sockaddr_in *v4 = (struct sockaddr_in *)sa;
 	struct sockaddr_in6 *v6 = (struct sockaddr_in6 *)sa;
 
-	if( v4->sin_family == AF_INET )
+	if (v4->sin_family == AF_INET)
 		return inet_ntop(AF_INET, &(v4->sin_addr), str, sizeof(str));
 	else
 		return inet_ntop(AF_INET6, &(v6->sin6_addr), str, sizeof(str));
@@ -64,7 +64,7 @@ int sa_rfc1918(void *sa)
 	struct sockaddr_in *v4 = (struct sockaddr_in *)sa;
 	unsigned long a = htonl(v4->sin_addr.s_addr);
 
-	if( v4->sin_family == AF_INET )
+	if (v4->sin_family == AF_INET)
 	{
 		return ((a >= 0x0A000000) && (a <= 0x0AFFFFFF)) ||
 		       ((a >= 0xAC100000) && (a <= 0xAC1FFFFF)) ||
@@ -80,22 +80,22 @@ char *strfind(char *haystack, int hslen, const char *needle, int ndlen)
 	int match = 0;
 	int i, j;
 
-	for( i = 0; i < hslen; i++ )
+	for (i = 0; i < hslen; i++)
 	{
-		if( haystack[i] == needle[0] )
+		if (haystack[i] == needle[0])
 		{
 			match = ((ndlen == 1) || ((i + ndlen) <= hslen));
 
-			for( j = 1; (j < ndlen) && ((i + j) < hslen); j++ )
+			for (j = 1; (j < ndlen) && ((i + j) < hslen); j++)
 			{
-				if( haystack[i+j] != needle[j] )
+				if (haystack[i+j] != needle[j])
 				{
 					match = 0;
 					break;
 				}
 			}
 
-			if( match )
+			if (match)
 				return &haystack[i];
 		}
 	}
@@ -160,7 +160,7 @@ int uh_tcp_peek(struct client *cl, char *buf, int len)
 	int sz = uh_tcp_recv(cl, buf, len);
 
 	/* store received data in peek buffer */
-	if( sz > 0 )
+	if (sz > 0)
 	{
 		cl->peeklen = sz;
 		memcpy(cl->peekbuf, buf, sz);
@@ -220,7 +220,8 @@ int uh_tcp_recv(struct client *cl, char *buf, int len)
 }
 
 
-int uh_http_sendhf(struct client *cl, int code, const char *summary, const char *fmt, ...)
+int uh_http_sendhf(struct client *cl, int code, const char *summary,
+				   const char *fmt, ...)
 {
 	va_list ap;
 
@@ -253,10 +254,10 @@ int uh_http_sendc(struct client *cl, const char *data, int len)
 	char chunk[8];
 	int clen;
 
-	if( len == -1 )
+	if (len == -1)
 		len = strlen(data);
 
-	if( len > 0 )
+	if (len > 0)
 	{
 		clen = snprintf(chunk, sizeof(chunk), "%X\r\n", len);
 		ensure_ret(uh_tcp_send(cl, chunk, clen));
@@ -271,9 +272,9 @@ int uh_http_sendc(struct client *cl, const char *data, int len)
 	return 0;
 }
 
-int uh_http_sendf(
-	struct client *cl, struct http_request *req, const char *fmt, ...
-) {
+int uh_http_sendf(struct client *cl, struct http_request *req,
+				  const char *fmt, ...)
+{
 	va_list ap;
 	char buffer[UH_LIMIT_MSGHEAD];
 	int len;
@@ -282,23 +283,23 @@ int uh_http_sendf(
 	len = vsnprintf(buffer, sizeof(buffer), fmt, ap);
 	va_end(ap);
 
-	if( (req != NULL) && (req->version > 1.0) )
+	if ((req != NULL) && (req->version > 1.0))
 		ensure_ret(uh_http_sendc(cl, buffer, len));
-	else if( len > 0 )
+	else if (len > 0)
 		ensure_ret(uh_tcp_send(cl, buffer, len));
 
 	return 0;
 }
 
-int uh_http_send(
-	struct client *cl, struct http_request *req, const char *buf, int len
-) {
-	if( len < 0 )
+int uh_http_send(struct client *cl, struct http_request *req,
+				 const char *buf, int len)
+{
+	if (len < 0)
 		len = strlen(buf);
 
-	if( (req != NULL) && (req->version > 1.0) )
+	if ((req != NULL) && (req->version > 1.0))
 		ensure_ret(uh_http_sendc(cl, buf, len));
-	else if( len > 0 )
+	else if (len > 0)
 		ensure_ret(uh_tcp_send(cl, buf, len));
 
 	return 0;
@@ -318,11 +319,11 @@ int uh_urldecode(char *buf, int blen, const char *src, int slen)
 		(((x) <= 'F') ? ((x) - 'A' + 10) : \
 			((x) - 'a' + 10)))
 
-	for( i = 0; (i < slen) && (len < blen); i++ )
+	for (i = 0; (i < slen) && (len < blen); i++)
 	{
-		if( src[i] == '%' )
+		if (src[i] == '%')
 		{
-			if( ((i+2) < slen) && isxdigit(src[i+1]) && isxdigit(src[i+2]) )
+			if (((i+2) < slen) && isxdigit(src[i+1]) && isxdigit(src[i+2]))
 			{
 				buf[len++] = (char)(16 * hex(src[i+1]) + hex(src[i+2]));
 				i += 2;
@@ -358,14 +359,14 @@ int uh_urlencode(char *buf, int blen, const char *src, int slen)
 	int len = 0;
 	const char hex[] = "0123456789abcdef";
 
-	for( i = 0; (i < slen) && (len < blen); i++ )
+	for (i = 0; (i < slen) && (len < blen); i++)
 	{
 		if( isalnum(src[i]) || (src[i] == '-') || (src[i] == '_') ||
 		    (src[i] == '.') || (src[i] == '~') )
 		{
 			buf[len++] = src[i];
 		}
-		else if( (len+3) <= blen )
+		else if ((len+3) <= blen)
 		{
 			buf[len++] = '%';
 			buf[len++] = hex[(src[i] >> 4) & 15];
@@ -390,30 +391,30 @@ int uh_b64decode(char *buf, int blen, const unsigned char *src, int slen)
 	unsigned int cout = 0;
 
 
-	for( i = 0; (i <= slen) && (src[i] != 0); i++ )
+	for (i = 0; (i <= slen) && (src[i] != 0); i++)
 	{
 		cin = src[i];
 
-		if( (cin >= '0') && (cin <= '9') )
+		if ((cin >= '0') && (cin <= '9'))
 			cin = cin - '0' + 52;
-		else if( (cin >= 'A') && (cin <= 'Z') )
+		else if ((cin >= 'A') && (cin <= 'Z'))
 			cin = cin - 'A';
-		else if( (cin >= 'a') && (cin <= 'z') )
+		else if ((cin >= 'a') && (cin <= 'z'))
 			cin = cin - 'a' + 26;
-		else if( cin == '+' )
+		else if (cin == '+')
 			cin = 62;
-		else if( cin == '/' )
+		else if (cin == '/')
 			cin = 63;
-		else if( cin == '=' )
+		else if (cin == '=')
 			cin = 0;
 		else
 			continue;
 
 		cout = (cout << 6) | cin;
 
-		if( (i % 4) == 3 )
+		if ((i % 4) == 3)
 		{
-			if( (len + 3) < blen )
+			if ((len + 3) < blen)
 			{
 				buf[len++] = (char)(cout >> 16);
 				buf[len++] = (char)(cout >> 8);
@@ -440,7 +441,7 @@ static char * canonpath(const char *path, char *path_resolved)
 
 
 	/* relative -> absolute */
-	if( *path != '/' )
+	if (*path != '/')
 	{
 		getcwd(path_copy, PATH_MAX);
 		strncat(path_copy, "/", PATH_MAX - strlen(path_copy));
@@ -452,32 +453,32 @@ static char * canonpath(const char *path, char *path_resolved)
 	}
 
 	/* normalize */
-	while( (*path_cpy != '\0') && (path_cpy < (path_copy + PATH_MAX - 2)) )
+	while ((*path_cpy != '\0') && (path_cpy < (path_copy + PATH_MAX - 2)))
 	{
-		if( *path_cpy == '/' )
+		if (*path_cpy == '/')
 		{
 			/* skip repeating / */
-			if( path_cpy[1] == '/' )
+			if (path_cpy[1] == '/')
 			{
 				path_cpy++;
 				continue;
 			}
 
 			/* /./ or /../ */
-			else if( path_cpy[1] == '.' )
+			else if (path_cpy[1] == '.')
 			{
 				/* skip /./ */
-				if( (path_cpy[2] == '/') || (path_cpy[2] == '\0') )
+				if ((path_cpy[2] == '/') || (path_cpy[2] == '\0'))
 				{
 					path_cpy += 2;
 					continue;
 				}
 
 				/* collapse /x/../ */
-				else if( (path_cpy[2] == '.') &&
-				         ((path_cpy[3] == '/') || (path_cpy[3] == '\0'))
-				) {
-					while( (path_res > path_resolved) && (*--path_res != '/') )
+				else if ((path_cpy[2] == '.') &&
+						 ((path_cpy[3] == '/') || (path_cpy[3] == '\0')))
+				{
+					while ((path_res > path_resolved) && (*--path_res != '/'))
 						;
 
 					path_cpy += 3;
@@ -490,15 +491,15 @@ static char * canonpath(const char *path, char *path_resolved)
 	}
 
 	/* remove trailing slash if not root / */
-	if( (path_res > (path_resolved+1)) && (path_res[-1] == '/') )
+	if ((path_res > (path_resolved+1)) && (path_res[-1] == '/'))
 		path_res--;
-	else if( path_res == path_resolved )
+	else if (path_res == path_resolved)
 		*path_res++ = '/';
 
 	*path_res = '\0';
 
 	/* test access */
-	if( !stat(path_resolved, &s) && (s.st_mode & S_IROTH) )
+	if (!stat(path_resolved, &s) && (s.st_mode & S_IROTH))
 		return path_resolved;
 
 	return NULL;
@@ -523,7 +524,7 @@ struct path_info * uh_path_lookup(struct client *cl, const char *url)
 	struct stat s;
 
 	/* back out early if url is undefined */
-	if ( url == NULL )
+	if (url == NULL)
 		return NULL;
 
 	memset(path_phys, 0, sizeof(path_phys));
@@ -533,46 +534,50 @@ struct path_info * uh_path_lookup(struct client *cl, const char *url)
 
 	/* copy docroot */
 	memcpy(buffer, docroot,
-		min(strlen(docroot), sizeof(buffer) - 1));
+		   min(strlen(docroot), sizeof(buffer) - 1));
 
 	/* separate query string from url */
-	if( (pathptr = strchr(url, '?')) != NULL )
+	if ((pathptr = strchr(url, '?')) != NULL)
 	{
 		p.query = pathptr[1] ? pathptr + 1 : NULL;
 
 		/* urldecode component w/o query */
-		if( pathptr > url )
-			if ( uh_urldecode(
-					&buffer[strlen(docroot)],
-					sizeof(buffer) - strlen(docroot) - 1,
-					url, pathptr - url ) < 0 )
+		if (pathptr > url)
+		{
+			if (uh_urldecode(&buffer[strlen(docroot)],
+							 sizeof(buffer) - strlen(docroot) - 1,
+							 url, pathptr - url ) < 0)
+			{
 				return NULL; /* bad URL */
+			}
+		}
 	}
 
 	/* no query string, decode all of url */
 	else
 	{
-		if ( uh_urldecode(
-				&buffer[strlen(docroot)],
-				sizeof(buffer) - strlen(docroot) - 1,
-				url, strlen(url) ) < 0 )
+		if (uh_urldecode(&buffer[strlen(docroot)],
+						 sizeof(buffer) - strlen(docroot) - 1,
+						 url, strlen(url) ) < 0)
+		{
 			return NULL; /* bad URL */
+		}
 	}
 
 	/* create canon path */
-	for( i = strlen(buffer), slash = (buffer[max(0, i-1)] == '/'); i >= 0; i-- )
+	for (i = strlen(buffer), slash = (buffer[max(0, i-1)] == '/'); i >= 0; i--)
 	{
-		if( (buffer[i] == 0) || (buffer[i] == '/') )
+		if ((buffer[i] == 0) || (buffer[i] == '/'))
 		{
 			memset(path_info, 0, sizeof(path_info));
 			memcpy(path_info, buffer, min(i + 1, sizeof(path_info) - 1));
 
-			if( no_sym ? realpath(path_info, path_phys)
-			           : canonpath(path_info, path_phys)
-			) {
+			if (no_sym ? realpath(path_info, path_phys)
+			           : canonpath(path_info, path_phys))
+			{
 				memset(path_info, 0, sizeof(path_info));
 				memcpy(path_info, &buffer[i],
-					min(strlen(buffer) - i, sizeof(path_info) - 1));
+					   min(strlen(buffer) - i, sizeof(path_info) - 1));
 
 				break;
 			}
@@ -580,18 +585,18 @@ struct path_info * uh_path_lookup(struct client *cl, const char *url)
 	}
 
 	/* check whether found path is within docroot */
-	if( strncmp(path_phys, docroot, strlen(docroot)) ||
-	    ((path_phys[strlen(docroot)] != 0) &&
-		 (path_phys[strlen(docroot)] != '/'))
-	) {
+	if (strncmp(path_phys, docroot, strlen(docroot)) ||
+		((path_phys[strlen(docroot)] != 0) &&
+		 (path_phys[strlen(docroot)] != '/')))
+	{
 		return NULL;
 	}
 
 	/* test current path */
-	if( ! stat(path_phys, &p.stat) )
+	if (!stat(path_phys, &p.stat))
 	{
 		/* is a regular file */
-		if( p.stat.st_mode & S_IFREG )
+		if (p.stat.st_mode & S_IFREG)
 		{
 			p.root = docroot;
 			p.phys = path_phys;
@@ -600,10 +605,10 @@ struct path_info * uh_path_lookup(struct client *cl, const char *url)
 		}
 
 		/* is a directory */
-		else if( (p.stat.st_mode & S_IFDIR) && !strlen(path_info) )
+		else if ((p.stat.st_mode & S_IFDIR) && !strlen(path_info))
 		{
 			/* ensure trailing slash */
-			if( path_phys[strlen(path_phys)-1] != '/' )
+			if (path_phys[strlen(path_phys)-1] != '/')
 				path_phys[strlen(path_phys)] = '/';
 
 			/* try to locate index file */
@@ -614,7 +619,7 @@ struct path_info * uh_path_lookup(struct client *cl, const char *url)
 			/* if requested url resolves to a directory and a trailing slash
 			   is missing in the request url, redirect the client to the same
 			   url with trailing slash appended */
-			if( !slash )
+			if (!slash)
 			{
 				uh_http_sendf(cl, NULL,
 					"HTTP/1.1 302 Found\r\n"
@@ -627,11 +632,11 @@ struct path_info * uh_path_lookup(struct client *cl, const char *url)
 
 				p.redirected = 1;
 			}
-			else if( cl->server->conf->index_file )
+			else if (cl->server->conf->index_file)
 			{
 				strncat(buffer, cl->server->conf->index_file, sizeof(buffer));
 
-				if( !stat(buffer, &s) && (s.st_mode & S_IFREG) )
+				if (!stat(buffer, &s) && (s.st_mode & S_IFREG))
 				{
 					memcpy(path_phys, buffer, sizeof(path_phys));
 					memcpy(&p.stat, &s, sizeof(p.stat));
@@ -639,11 +644,11 @@ struct path_info * uh_path_lookup(struct client *cl, const char *url)
 			}
 			else
 			{
-				for( i = 0; i < array_size(uh_index_files); i++ )
+				for (i = 0; i < array_size(uh_index_files); i++)
 				{
 					strncat(buffer, uh_index_files[i], sizeof(buffer));
 
-					if( !stat(buffer, &s) && (s.st_mode & S_IFREG) )
+					if (!stat(buffer, &s) && (s.st_mode & S_IFREG))
 					{
 						memcpy(path_phys, buffer, sizeof(path_phys));
 						memcpy(&p.stat, &s, sizeof(p.stat));
@@ -680,31 +685,31 @@ struct auth_realm * uh_auth_add(char *path, char *user, char *pass)
 		memset(new, 0, sizeof(struct auth_realm));
 
 		memcpy(new->path, path,
-			min(strlen(path), sizeof(new->path) - 1));
+			   min(strlen(path), sizeof(new->path) - 1));
 
 		memcpy(new->user, user,
-			min(strlen(user), sizeof(new->user) - 1));
+			   min(strlen(user), sizeof(new->user) - 1));
 
 		/* given password refers to a passwd entry */
-		if( (strlen(pass) > 3) && !strncmp(pass, "$p$", 3) )
+		if ((strlen(pass) > 3) && !strncmp(pass, "$p$", 3))
 		{
 #ifdef HAVE_SHADOW
 			/* try to resolve shadow entry */
-			if( ((spwd = getspnam(&pass[3])) != NULL) && spwd->sp_pwdp )
+			if (((spwd = getspnam(&pass[3])) != NULL) && spwd->sp_pwdp)
 			{
 				memcpy(new->pass, spwd->sp_pwdp,
-					min(strlen(spwd->sp_pwdp), sizeof(new->pass) - 1));
+					   min(strlen(spwd->sp_pwdp), sizeof(new->pass) - 1));
 			}
 
 			else
 #endif
 
 			/* try to resolve passwd entry */
-			if( ((pwd = getpwnam(&pass[3])) != NULL) && pwd->pw_passwd &&
-				(pwd->pw_passwd[0] != '!') && (pwd->pw_passwd[0] != 0)
-			) {
+			if (((pwd = getpwnam(&pass[3])) != NULL) && pwd->pw_passwd &&
+				(pwd->pw_passwd[0] != '!') && (pwd->pw_passwd[0] != 0))
+			{
 				memcpy(new->pass, pwd->pw_passwd,
-					min(strlen(pwd->pw_passwd), sizeof(new->pass) - 1));
+					   min(strlen(pwd->pw_passwd), sizeof(new->pass) - 1));
 			}
 		}
 
@@ -715,7 +720,7 @@ struct auth_realm * uh_auth_add(char *path, char *user, char *pass)
 				min(strlen(pass), sizeof(new->pass) - 1));
 		}
 
-		if( new->pass[0] )
+		if (new->pass[0])
 		{
 			new->next = uh_realms;
 			uh_realms = new;
@@ -729,9 +734,9 @@ struct auth_realm * uh_auth_add(char *path, char *user, char *pass)
 	return NULL;
 }
 
-int uh_auth_check(
-	struct client *cl, struct http_request *req, struct path_info *pi
-) {
+int uh_auth_check(struct client *cl, struct http_request *req,
+				  struct path_info *pi)
+{
 	int i, plen, rlen, protected;
 	char buffer[UH_LIMIT_MSGHEAD];
 	char *user = NULL;
@@ -743,11 +748,11 @@ int uh_auth_check(
 	protected = 0;
 
 	/* check whether at least one realm covers the requested url */
-	for( realm = uh_realms; realm; realm = realm->next )
+	for (realm = uh_realms; realm; realm = realm->next)
 	{
 		rlen = strlen(realm->path);
 
-		if( (plen >= rlen) && !strncasecmp(pi->name, realm->path, rlen) )
+		if ((plen >= rlen) && !strncasecmp(pi->name, realm->path, rlen))
 		{
 			req->realm = realm;
 			protected = 1;
@@ -756,21 +761,21 @@ int uh_auth_check(
 	}
 
 	/* requested resource is covered by a realm */
-	if( protected )
+	if (protected)
 	{
 		/* try to get client auth info */
 		foreach_header(i, req->headers)
 		{
-			if( !strcasecmp(req->headers[i], "Authorization") &&
+			if (!strcasecmp(req->headers[i], "Authorization") &&
 				(strlen(req->headers[i+1]) > 6) &&
-				!strncasecmp(req->headers[i+1], "Basic ", 6)
-			) {
+				!strncasecmp(req->headers[i+1], "Basic ", 6))
+			{
 				memset(buffer, 0, sizeof(buffer));
 				uh_b64decode(buffer, sizeof(buffer) - 1,
 					(unsigned char *) &req->headers[i+1][6],
 					strlen(req->headers[i+1]) - 6);
 
-				if( (pass = strchr(buffer, ':')) != NULL )
+				if ((pass = strchr(buffer, ':')) != NULL)
 				{
 					user = buffer;
 					*pass++ = 0;
@@ -781,24 +786,24 @@ int uh_auth_check(
 		}
 
 		/* have client auth */
-		if( user && pass )
+		if (user && pass)
 		{
 			/* find matching realm */
-			for( realm = uh_realms; realm; realm = realm->next )
+			for (realm = uh_realms; realm; realm = realm->next)
 			{
 				rlen = strlen(realm->path);
 
-				if( (plen >= rlen) &&
-				    !strncasecmp(pi->name, realm->path, rlen) &&
-				    !strcmp(user, realm->user)
-				) {
+				if ((plen >= rlen) &&
+					!strncasecmp(pi->name, realm->path, rlen) &&
+					!strcmp(user, realm->user))
+				{
 					req->realm = realm;
 					break;
 				}
 			}
 
 			/* found a realm matching the username */
-			if( realm )
+			if (realm)
 			{
 				/* check user pass */
 				if (!strcmp(pass, realm->pass) ||
@@ -832,7 +837,7 @@ struct listener * uh_listener_add(int sock, struct config *conf)
 	struct listener *new = NULL;
 	socklen_t sl;
 
-	if( (new = (struct listener *)malloc(sizeof(struct listener))) != NULL )
+	if ((new = (struct listener *)malloc(sizeof(struct listener))) != NULL)
 	{
 		memset(new, 0, sizeof(struct listener));
 
@@ -857,8 +862,8 @@ struct listener * uh_listener_lookup(int sock)
 {
 	struct listener *cur = NULL;
 
-	for( cur = uh_listeners; cur; cur = cur->next )
-		if( cur->socket == sock )
+	for (cur = uh_listeners; cur; cur = cur->next)
+		if (cur->socket == sock)
 			return cur;
 
 	return NULL;
@@ -870,7 +875,7 @@ struct client * uh_client_add(int sock, struct listener *serv)
 	struct client *new = NULL;
 	socklen_t sl;
 
-	if( (new = (struct client *)malloc(sizeof(struct client))) != NULL )
+	if ((new = (struct client *)malloc(sizeof(struct client))) != NULL)
 	{
 		memset(new, 0, sizeof(struct client));
 
@@ -898,8 +903,8 @@ struct client * uh_client_lookup(int sock)
 {
 	struct client *cur = NULL;
 
-	for( cur = uh_clients; cur; cur = cur->next )
-		if( cur->socket == sock )
+	for (cur = uh_clients; cur; cur = cur->next)
+		if (cur->socket == sock)
 			return cur;
 
 	return NULL;
@@ -910,11 +915,11 @@ void uh_client_remove(int sock)
 	struct client *cur = NULL;
 	struct client *prv = NULL;
 
-	for( cur = uh_clients; cur; prv = cur, cur = cur->next )
+	for (cur = uh_clients; cur; prv = cur, cur = cur->next)
 	{
-		if( cur->socket == sock )
+		if (cur->socket == sock)
 		{
-			if( prv )
+			if (prv)
 				prv->next = cur->next;
 			else
 				uh_clients = cur->next;
@@ -933,8 +938,7 @@ struct interpreter * uh_interpreter_add(const char *extn, const char *path)
 {
 	struct interpreter *new = NULL;
 
-	if( (new = (struct interpreter *)
-			malloc(sizeof(struct interpreter))) != NULL )
+	if ((new = (struct interpreter *)malloc(sizeof(struct interpreter))) != NULL)
 	{
 		memset(new, 0, sizeof(struct interpreter));
 
@@ -955,11 +959,11 @@ struct interpreter * uh_interpreter_lookup(const char *path)
 	struct interpreter *cur = NULL;
 	const char *e;
 
-	for( cur = uh_interpreters; cur; cur = cur->next )
+	for (cur = uh_interpreters; cur; cur = cur->next)
 	{
 		e = &path[max(strlen(path) - strlen(cur->extn), 0)];
 
-		if( !strcmp(e, cur->extn) )
+		if (!strcmp(e, cur->extn))
 			return cur;
 	}
 
diff --git a/package/uhttpd/src/uhttpd.c b/package/uhttpd/src/uhttpd.c
index 78e0b5d952..0592811386 100644
--- a/package/uhttpd/src/uhttpd.c
+++ b/package/uhttpd/src/uhttpd.c
@@ -44,7 +44,7 @@ static void uh_sigterm(int sig)
 
 static void uh_sigchld(int sig)
 {
-	while( waitpid(-1, NULL, WNOHANG) > 0 ) { }
+	while (waitpid(-1, NULL, WNOHANG) > 0) { }
 }
 
 static void uh_config_parse(struct config *conf)
@@ -58,56 +58,64 @@ static void uh_config_parse(struct config *conf)
 	const char *path = conf->file ? conf->file : "/etc/httpd.conf";
 
 
-	if( (c = fopen(path, "r")) != NULL )
+	if ((c = fopen(path, "r")) != NULL)
 	{
 		memset(line, 0, sizeof(line));
 
-		while( fgets(line, sizeof(line) - 1, c) )
+		while (fgets(line, sizeof(line) - 1, c))
 		{
-			if( (line[0] == '/') && (strchr(line, ':') != NULL) )
+			if ((line[0] == '/') && (strchr(line, ':') != NULL))
 			{
-				if( !(col1 = strchr(line, ':')) || (*col1++ = 0) ||
+				if (!(col1 = strchr(line, ':')) || (*col1++ = 0) ||
 				    !(col2 = strchr(col1, ':')) || (*col2++ = 0) ||
-					!(eol = strchr(col2, '\n')) || (*eol++  = 0) )
-						continue;
+					!(eol = strchr(col2, '\n')) || (*eol++  = 0))
+				{
+					continue;
+				}
 
-				if( !uh_auth_add(line, col1, col2) )
+				if (!uh_auth_add(line, col1, col2))
 				{
 					fprintf(stderr,
-						"Notice: No password set for user %s, ignoring "
-						"authentication on %s\n", col1, line
+							"Notice: No password set for user %s, ignoring "
+							"authentication on %s\n", col1, line
 					);
 				}
 			}
-			else if( !strncmp(line, "I:", 2) )
+			else if (!strncmp(line, "I:", 2))
 			{
-				if( !(col1 = strchr(line, ':')) || (*col1++ = 0) ||
-				    !(eol = strchr(col1, '\n')) || (*eol++  = 0) )
-				    	continue;
+				if (!(col1 = strchr(line, ':')) || (*col1++ = 0) ||
+				    !(eol = strchr(col1, '\n')) || (*eol++  = 0))
+				{
+				   	continue;
+				}
 
 				conf->index_file = strdup(col1);
 			}
-			else if( !strncmp(line, "E404:", 5) )
+			else if (!strncmp(line, "E404:", 5))
 			{
-				if( !(col1 = strchr(line, ':')) || (*col1++ = 0) ||
-				    !(eol = strchr(col1, '\n')) || (*eol++  = 0) )
-						continue;
+				if (!(col1 = strchr(line, ':')) || (*col1++ = 0) ||
+				    !(eol = strchr(col1, '\n')) || (*eol++  = 0))
+				{
+					continue;
+				}
 
 				conf->error_handler = strdup(col1);
 			}
 #ifdef HAVE_CGI
-			else if( (line[0] == '*') && (strchr(line, ':') != NULL) )
+			else if ((line[0] == '*') && (strchr(line, ':') != NULL))
 			{
-				if( !(col1 = strchr(line, '*')) || (*col1++ = 0) ||
+				if (!(col1 = strchr(line, '*')) || (*col1++ = 0) ||
 				    !(col2 = strchr(col1, ':')) || (*col2++ = 0) ||
-				    !(eol = strchr(col2, '\n')) || (*eol++  = 0) )
-						continue;
+				    !(eol = strchr(col2, '\n')) || (*eol++  = 0))
+				{
+					continue;
+				}
 
-				if( !uh_interpreter_add(col1, col2) )
+				if (!uh_interpreter_add(col1, col2))
 				{
 					fprintf(stderr,
-						"Unable to add interpreter %s for extension %s: "
-						"Out of memory\n", col2, col1
+							"Unable to add interpreter %s for extension %s: "
+							"Out of memory\n", col2, col1
 					);
 				}
 			}
@@ -118,10 +126,11 @@ static void uh_config_parse(struct config *conf)
 	}
 }
 
-static int uh_socket_bind(
-	fd_set *serv_fds, int *max_fd, const char *host, const char *port,
-	struct addrinfo *hints, int do_tls, struct config *conf
-) {
+static int uh_socket_bind(fd_set *serv_fds, int *max_fd,
+						  const char *host, const char *port,
+						  struct addrinfo *hints, int do_tls,
+						  struct config *conf)
+{
 	int sock = -1;
 	int yes = 1;
 	int status;
@@ -132,39 +141,39 @@ static int uh_socket_bind(
 	struct listener *l = NULL;
 	struct addrinfo *addrs = NULL, *p = NULL;
 
-	if( (status = getaddrinfo(host, port, hints, &addrs)) != 0 )
+	if ((status = getaddrinfo(host, port, hints, &addrs)) != 0)
 	{
 		fprintf(stderr, "getaddrinfo(): %s\n", gai_strerror(status));
 	}
 
 	/* try to bind a new socket to each found address */
-	for( p = addrs; p; p = p->ai_next )
+	for (p = addrs; p; p = p->ai_next)
 	{
 		/* get the socket */
-		if( (sock = socket(p->ai_family, p->ai_socktype, p->ai_protocol)) == -1 )
+		if ((sock = socket(p->ai_family, p->ai_socktype, p->ai_protocol)) == -1)
 		{
 			perror("socket()");
 			goto error;
 		}
 
 		/* "address already in use" */
-		if( setsockopt(sock, SOL_SOCKET, SO_REUSEADDR, &yes, sizeof(yes)) )
+		if (setsockopt(sock, SOL_SOCKET, SO_REUSEADDR, &yes, sizeof(yes)))
 		{
 			perror("setsockopt()");
 			goto error;
 		}
 
 		/* TCP keep-alive */
-		if( conf->tcp_keepalive > 0 )
+		if (conf->tcp_keepalive > 0)
 		{
 			tcp_ka_idl = 1;
 			tcp_ka_cnt = 3;
 			tcp_ka_int = conf->tcp_keepalive;
 
-			if( setsockopt(sock, SOL_SOCKET, SO_KEEPALIVE, &yes, sizeof(yes)) ||
+			if (setsockopt(sock, SOL_SOCKET, SO_KEEPALIVE, &yes, sizeof(yes)) ||
 			    setsockopt(sock, SOL_TCP, TCP_KEEPIDLE,  &tcp_ka_idl, sizeof(tcp_ka_idl)) ||
 			    setsockopt(sock, SOL_TCP, TCP_KEEPINTVL, &tcp_ka_int, sizeof(tcp_ka_int)) ||
-			    setsockopt(sock, SOL_TCP, TCP_KEEPCNT,   &tcp_ka_cnt, sizeof(tcp_ka_cnt)) )
+			    setsockopt(sock, SOL_TCP, TCP_KEEPCNT,   &tcp_ka_cnt, sizeof(tcp_ka_cnt)))
 			{
 			    fprintf(stderr, "Notice: Unable to enable TCP keep-alive: %s\n",
 			    	strerror(errno));
@@ -172,9 +181,9 @@ static int uh_socket_bind(
 		}
 
 		/* required to get parallel v4 + v6 working */
-		if( p->ai_family == AF_INET6 )
+		if (p->ai_family == AF_INET6)
 		{
-			if( setsockopt(sock, IPPROTO_IPV6, IPV6_V6ONLY, &yes, sizeof(yes)) == -1 )
+			if (setsockopt(sock, IPPROTO_IPV6, IPV6_V6ONLY, &yes, sizeof(yes)) == -1)
 			{
 				perror("setsockopt()");
 				goto error;
@@ -182,21 +191,21 @@ static int uh_socket_bind(
 		}
 
 		/* bind */
-		if( bind(sock, p->ai_addr, p->ai_addrlen) == -1 )
+		if (bind(sock, p->ai_addr, p->ai_addrlen) == -1)
 		{
 			perror("bind()");
 			goto error;
 		}
 
 		/* listen */
-		if( listen(sock, UH_LIMIT_CLIENTS) == -1 )
+		if (listen(sock, UH_LIMIT_CLIENTS) == -1)
 		{
 			perror("listen()");
 			goto error;
 		}
 
 		/* add listener to global list */
-		if( ! (l = uh_listener_add(sock, conf)) )
+		if (!(l = uh_listener_add(sock, conf)))
 		{
 			fprintf(stderr, "uh_listener_add(): Failed to allocate memory\n");
 			goto error;
@@ -216,7 +225,7 @@ static int uh_socket_bind(
 		continue;
 
 		error:
-		if( sock > 0 )
+		if (sock > 0)
 			close(sock);
 	}
 
@@ -225,7 +234,8 @@ static int uh_socket_bind(
 	return bound;
 }
 
-static struct http_request * uh_http_header_parse(struct client *cl, char *buffer, int buflen)
+static struct http_request * uh_http_header_parse(struct client *cl,
+												  char *buffer, int buflen)
 {
 	char *method  = &buffer[0];
 	char *path    = NULL;
@@ -244,7 +254,7 @@ static struct http_request * uh_http_header_parse(struct client *cl, char *buffe
 
 
 	/* terminate initial header line */
-	if( (headers = strfind(buffer, buflen, "\r\n", 2)) != NULL )
+	if ((headers = strfind(buffer, buflen, "\r\n", 2)) != NULL)
 	{
 		buffer[buflen-1] = 0;
 
@@ -252,16 +262,16 @@ static struct http_request * uh_http_header_parse(struct client *cl, char *buffe
 		*headers++ = 0;
 
 		/* find request path */
-		if( (path = strchr(buffer, ' ')) != NULL )
+		if ((path = strchr(buffer, ' ')) != NULL)
 			*path++ = 0;
 
 		/* find http version */
-		if( (path != NULL) && ((version = strchr(path, ' ')) != NULL) )
+		if ((path != NULL) && ((version = strchr(path, ' ')) != NULL))
 			*version++ = 0;
 
 
 		/* check method */
-		if( strcmp(method, "GET") && strcmp(method, "HEAD") && strcmp(method, "POST") )
+		if (strcmp(method, "GET") && strcmp(method, "HEAD") && strcmp(method, "POST"))
 		{
 			/* invalid method */
 			uh_http_response(cl, 405, "Method Not Allowed");
@@ -286,7 +296,7 @@ static struct http_request * uh_http_header_parse(struct client *cl, char *buffe
 		}
 
 		/* check path */
-		if( !path || !strlen(path) )
+		if (!path || !strlen(path))
 		{
 			/* malformed request */
 			uh_http_response(cl, 400, "Bad Request");
@@ -298,8 +308,8 @@ static struct http_request * uh_http_header_parse(struct client *cl, char *buffe
 		}
 
 		/* check version */
-		if( (version == NULL) || (strcmp(version, "HTTP/0.9") &&
-		    strcmp(version, "HTTP/1.0") && strcmp(version, "HTTP/1.1")) )
+		if ((version == NULL) || (strcmp(version, "HTTP/0.9") &&
+		    strcmp(version, "HTTP/1.0") && strcmp(version, "HTTP/1.1")))
 		{
 			/* unsupported version */
 			uh_http_response(cl, 400, "Bad Request");
@@ -312,15 +322,15 @@ static struct http_request * uh_http_header_parse(struct client *cl, char *buffe
 
 
 		/* process header fields */
-		for( i = (int)(headers - buffer); i < buflen; i++ )
+		for (i = (int)(headers - buffer); i < buflen; i++)
 		{
 			/* found eol and have name + value, push out header tuple */
-			if( hdrname && hdrdata && (buffer[i] == '\r' || buffer[i] == '\n') )
+			if (hdrname && hdrdata && (buffer[i] == '\r' || buffer[i] == '\n'))
 			{
 				buffer[i] = 0;
 
 				/* store */
-				if( (hdrcount + 1) < array_size(req.headers) )
+				if ((hdrcount + 1) < array_size(req.headers))
 				{
 					req.headers[hdrcount++] = hdrname;
 					req.headers[hdrcount++] = hdrdata;
@@ -337,9 +347,9 @@ static struct http_request * uh_http_header_parse(struct client *cl, char *buffe
 			}
 
 			/* have name but no value and found a colon, start of value */
-			else if( hdrname && !hdrdata &&
-			    ((i+1) < buflen) && (buffer[i] == ':')
-			) {
+			else if (hdrname && !hdrdata &&
+					 ((i+1) < buflen) && (buffer[i] == ':'))
+			{
 				buffer[i] = 0;
 				hdrdata = &buffer[i+1];
 
@@ -348,7 +358,7 @@ static struct http_request * uh_http_header_parse(struct client *cl, char *buffe
 			}
 
 			/* have no name and found [A-Za-z], start of name */
-			else if( !hdrname && isalpha(buffer[i]) )
+			else if (!hdrname && isalpha(buffer[i]))
 			{
 				hdrname = &buffer[i];
 			}
@@ -380,7 +390,7 @@ static struct http_request * uh_http_header_recv(struct client *cl)
 
 	memset(buffer, 0, sizeof(buffer));
 
-	while( blen > 0 )
+	while (blen > 0)
 	{
 		FD_ZERO(&reader);
 		FD_SET(cl->socket, &reader);
@@ -390,12 +400,12 @@ static struct http_request * uh_http_header_recv(struct client *cl)
 		timeout.tv_usec = 100000;
 
 		/* check whether fd is readable */
-		if( select(cl->socket + 1, &reader, NULL, NULL, &timeout) > 0 )
+		if (select(cl->socket + 1, &reader, NULL, NULL, &timeout) > 0)
 		{
 			/* receive data */
 			ensure_out(rlen = uh_tcp_peek(cl, bufptr, blen));
 
-			if( (idxptr = strfind(buffer, sizeof(buffer), "\r\n\r\n", 4)) )
+			if ((idxptr = strfind(buffer, sizeof(buffer), "\r\n\r\n", 4)))
 			{
 				ensure_out(rlen = uh_tcp_recv(cl, bufptr,
 					(int)(idxptr - bufptr) + 4));
@@ -410,7 +420,7 @@ static struct http_request * uh_http_header_recv(struct client *cl)
 				ensure_out(rlen = uh_tcp_recv(cl, bufptr, rlen));
 
 				/* unexpected eof - #7904 */
-				if( rlen == 0 )
+				if (rlen == 0)
 					return NULL;
 
 				blen -= rlen;
@@ -434,11 +444,11 @@ out:
 #if defined(HAVE_LUA) || defined(HAVE_CGI)
 static int uh_path_match(const char *prefix, const char *url)
 {
-	if( (strstr(url, prefix) == url) &&
-	    ((prefix[strlen(prefix)-1] == '/') ||
+	if ((strstr(url, prefix) == url) &&
+		((prefix[strlen(prefix)-1] == '/') ||
 		 (strlen(url) == strlen(prefix))   ||
-		 (url[strlen(prefix)] == '/'))
-	) {
+		 (url[strlen(prefix)] == '/')))
+	{
 		return 1;
 	}
 
@@ -446,14 +456,14 @@ static int uh_path_match(const char *prefix, const char *url)
 }
 #endif
 
-static void uh_dispatch_request(
-	struct client *cl, struct http_request *req, struct path_info *pin
-) {
+static void uh_dispatch_request(struct client *cl, struct http_request *req,
+								struct path_info *pin)
+{
 #ifdef HAVE_CGI
 	struct interpreter *ipr = NULL;
 
-	if( uh_path_match(cl->server->conf->cgi_prefix, pin->name) ||
-		(ipr = uh_interpreter_lookup(pin->phys)) )
+	if (uh_path_match(cl->server->conf->cgi_prefix, pin->name) ||
+		(ipr = uh_interpreter_lookup(pin->phys)))
 	{
 		uh_cgi_request(cl, req, pin, ipr);
 	}
@@ -485,42 +495,42 @@ static void uh_mainloop(struct config *conf, fd_set serv_fds, int max_fd)
 	used_fds = serv_fds;
 
 	/* loop */
-	while(run)
+	while (run)
 	{
 		/* create a working copy of the used fd set */
 		read_fds = used_fds;
 
 		/* sleep until socket activity */
-		if( select(max_fd + 1, &read_fds, NULL, NULL, NULL) == -1 )
+		if (select(max_fd + 1, &read_fds, NULL, NULL, NULL) == -1)
 		{
 			perror("select()");
 			exit(1);
 		}
 
 		/* run through the existing connections looking for data to be read */
-		for( cur_fd = 0; cur_fd <= max_fd; cur_fd++ )
+		for (cur_fd = 0; cur_fd <= max_fd; cur_fd++)
 		{
 			/* is a socket managed by us */
-			if( FD_ISSET(cur_fd, &read_fds) )
+			if (FD_ISSET(cur_fd, &read_fds))
 			{
 				/* is one of our listen sockets */
-				if( FD_ISSET(cur_fd, &serv_fds) )
+				if (FD_ISSET(cur_fd, &serv_fds))
 				{
 					/* handle new connections */
-					if( (new_fd = accept(cur_fd, NULL, 0)) != -1 )
+					if ((new_fd = accept(cur_fd, NULL, 0)) != -1)
 					{
 						/* add to global client list */
-						if( (cl = uh_client_add(new_fd, uh_listener_lookup(cur_fd))) != NULL )
+						if ((cl = uh_client_add(new_fd, uh_listener_lookup(cur_fd))) != NULL)
 						{
 #ifdef HAVE_TLS
 							/* setup client tls context */
-							if( conf->tls )
+							if (conf->tls)
 							{
-								if( conf->tls_accept(cl) < 1 )
+								if (conf->tls_accept(cl) < 1)
 								{
 									fprintf(stderr,
-										"tls_accept failed, "
-										"connection dropped\n");
+											"tls_accept failed, "
+											"connection dropped\n");
 
 									/* close client socket */
 									close(new_fd);
@@ -543,7 +553,8 @@ static void uh_mainloop(struct config *conf, fd_set serv_fds, int max_fd)
 						else
 						{
 							fprintf(stderr,
-								"uh_client_add(): Cannot allocate memory\n");
+									"uh_client_add(): "
+									"Cannot allocate memory\n");
 
 							close(new_fd);
 						}
@@ -553,43 +564,43 @@ static void uh_mainloop(struct config *conf, fd_set serv_fds, int max_fd)
 				/* is a client socket */
 				else
 				{
-					if( ! (cl = uh_client_lookup(cur_fd)) )
+					if (!(cl = uh_client_lookup(cur_fd)))
 					{
 						/* this should not happen! */
 						fprintf(stderr,
-							"uh_client_lookup(): No entry for fd %i!\n",
-							cur_fd);
+								"uh_client_lookup(): No entry for fd %i!\n",
+								cur_fd);
 
 						goto cleanup;
 					}
 
 					/* parse message header */
-					if( (req = uh_http_header_recv(cl)) != NULL )
+					if ((req = uh_http_header_recv(cl)) != NULL)
 					{
 						/* RFC1918 filtering required? */
-						if( conf->rfc1918_filter &&
+						if (conf->rfc1918_filter &&
 						    sa_rfc1918(&cl->peeraddr) &&
-						    !sa_rfc1918(&cl->servaddr) )
+						    !sa_rfc1918(&cl->servaddr))
 						{
 							uh_http_sendhf(cl, 403, "Forbidden",
-								"Rejected request from RFC1918 IP "
-								"to public server address");
+										   "Rejected request from RFC1918 IP "
+										   "to public server address");
 						}
 						else
 #ifdef HAVE_LUA
 						/* Lua request? */
-						if( conf->lua_state &&
-						    uh_path_match(conf->lua_prefix, req->url) )
+						if (conf->lua_state &&
+							uh_path_match(conf->lua_prefix, req->url))
 						{
 							conf->lua_request(cl, req, conf->lua_state);
 						}
 						else
 #endif
 						/* dispatch request */
-						if( (pin = uh_path_lookup(cl, req->url)) != NULL )
+						if ((pin = uh_path_lookup(cl, req->url)) != NULL)
 						{
 							/* auth ok? */
-							if( !pin->redirected && uh_auth_check(cl, req, pin) )
+							if (!pin->redirected && uh_auth_check(cl, req, pin))
 								uh_dispatch_request(cl, req, pin);
 						}
 
@@ -599,7 +610,7 @@ static void uh_mainloop(struct config *conf, fd_set serv_fds, int max_fd)
 							/* Try to invoke an error handler */
 							pin = uh_path_lookup(cl, conf->error_handler);
 
-							if( pin && uh_auth_check(cl, req, pin) )
+							if (pin && uh_auth_check(cl, req, pin))
 							{
 								req->redirect_status = 404;
 								uh_dispatch_request(cl, req, pin);
@@ -614,7 +625,7 @@ static void uh_mainloop(struct config *conf, fd_set serv_fds, int max_fd)
 
 #ifdef HAVE_TLS
 					/* free client tls context */
-					if( conf->tls )
+					if (conf->tls)
 						conf->tls_close(cl);
 #endif
 
@@ -633,7 +644,7 @@ static void uh_mainloop(struct config *conf, fd_set serv_fds, int max_fd)
 
 #ifdef HAVE_LUA
 	/* destroy the Lua state */
-	if( conf->lua_state != NULL )
+	if (conf->lua_state != NULL)
 		conf->lua_close(conf->lua_state);
 #endif
 }
@@ -645,15 +656,15 @@ static inline int uh_inittls(struct config *conf)
 	void *lib;
 
 	/* already loaded */
-	if( conf->tls != NULL )
+	if (conf->tls != NULL)
 		return 0;
 
 	/* load TLS plugin */
-	if( ! (lib = dlopen("uhttpd_tls.so", RTLD_LAZY | RTLD_GLOBAL)) )
+	if (!(lib = dlopen("uhttpd_tls.so", RTLD_LAZY | RTLD_GLOBAL)))
 	{
 		fprintf(stderr,
-			"Notice: Unable to load TLS plugin - disabling SSL support! "
-			"(Reason: %s)\n", dlerror()
+				"Notice: Unable to load TLS plugin - disabling SSL support! "
+				"(Reason: %s)\n", dlerror()
 		);
 
 		return 1;
@@ -661,24 +672,24 @@ static inline int uh_inittls(struct config *conf)
 	else
 	{
 		/* resolve functions */
-		if( !(conf->tls_init   = dlsym(lib, "uh_tls_ctx_init"))      ||
+		if (!(conf->tls_init   = dlsym(lib, "uh_tls_ctx_init"))      ||
 		    !(conf->tls_cert   = dlsym(lib, "uh_tls_ctx_cert"))      ||
 		    !(conf->tls_key    = dlsym(lib, "uh_tls_ctx_key"))       ||
 		    !(conf->tls_free   = dlsym(lib, "uh_tls_ctx_free"))      ||
 		    !(conf->tls_accept = dlsym(lib, "uh_tls_client_accept")) ||
 		    !(conf->tls_close  = dlsym(lib, "uh_tls_client_close"))  ||
 		    !(conf->tls_recv   = dlsym(lib, "uh_tls_client_recv"))   ||
-		    !(conf->tls_send   = dlsym(lib, "uh_tls_client_send"))
-		) {
+		    !(conf->tls_send   = dlsym(lib, "uh_tls_client_send")))
+		{
 			fprintf(stderr,
-				"Error: Failed to lookup required symbols "
-				"in TLS plugin: %s\n", dlerror()
+					"Error: Failed to lookup required symbols "
+					"in TLS plugin: %s\n", dlerror()
 			);
 			exit(1);
 		}
 
 		/* init SSL context */
-		if( ! (conf->tls = conf->tls_init()) )
+		if (!(conf->tls = conf->tls_init()))
 		{
 			fprintf(stderr, "Error: Failed to initalize SSL context\n");
 			exit(1);
@@ -755,17 +766,17 @@ int main (int argc, char **argv)
 	memset(bind, 0, sizeof(bind));
 
 
-	while( (opt = getopt(argc, argv,
-		"fSDRC:K:E:I:p:s:h:c:l:L:d:r:m:x:i:t:T:A:")) > 0
-	) {
+	while ((opt = getopt(argc, argv,
+						 "fSDRC:K:E:I:p:s:h:c:l:L:d:r:m:x:i:t:T:A:")) > 0)
+	{
 		switch(opt)
 		{
 			/* [addr:]port */
 			case 'p':
 			case 's':
-				if( (port = strrchr(optarg, ':')) != NULL )
+				if ((port = strrchr(optarg, ':')) != NULL)
 				{
-					if( (optarg[0] == '[') && (port > optarg) && (port[-1] == ']') )
+					if ((optarg[0] == '[') && (port > optarg) && (port[-1] == ']'))
 						memcpy(bind, optarg + 1,
 							min(sizeof(bind), (int)(port - optarg) - 2));
 					else
@@ -780,9 +791,9 @@ int main (int argc, char **argv)
 				}
 
 #ifdef HAVE_TLS
-				if( opt == 's' )
+				if (opt == 's')
 				{
-					if( uh_inittls(&conf) )
+					if (uh_inittls(&conf))
 					{
 						fprintf(stderr,
 							"Notice: TLS support is disabled, "
@@ -796,10 +807,9 @@ int main (int argc, char **argv)
 #endif
 
 				/* bind sockets */
-				bound += uh_socket_bind(
-					&serv_fds, &max_fd, bind[0] ? bind : NULL, port,
-					&hints,	(opt == 's'), &conf
-				);
+				bound += uh_socket_bind(&serv_fds, &max_fd,
+										bind[0] ? bind : NULL,
+										port, &hints, (opt == 's'), &conf);
 
 				memset(bind, 0, sizeof(bind));
 				break;
@@ -807,12 +817,12 @@ int main (int argc, char **argv)
 #ifdef HAVE_TLS
 			/* certificate */
 			case 'C':
-				if( !uh_inittls(&conf) )
+				if (!uh_inittls(&conf))
 				{
-					if( conf.tls_cert(conf.tls, optarg) < 1 )
+					if (conf.tls_cert(conf.tls, optarg) < 1)
 					{
 						fprintf(stderr,
-							"Error: Invalid certificate file given\n");
+								"Error: Invalid certificate file given\n");
 						exit(1);
 					}
 
@@ -823,12 +833,12 @@ int main (int argc, char **argv)
 
 			/* key */
 			case 'K':
-				if( !uh_inittls(&conf) )
+				if (!uh_inittls(&conf))
 				{
-					if( conf.tls_key(conf.tls, optarg) < 1 )
+					if (conf.tls_key(conf.tls, optarg) < 1)
 					{
 						fprintf(stderr,
-							"Error: Invalid private key file given\n");
+								"Error: Invalid private key file given\n");
 						exit(1);
 					}
 
@@ -840,20 +850,20 @@ int main (int argc, char **argv)
 
 			/* docroot */
 			case 'h':
-				if( ! realpath(optarg, conf.docroot) )
+				if (! realpath(optarg, conf.docroot))
 				{
 					fprintf(stderr, "Error: Invalid directory %s: %s\n",
-						optarg, strerror(errno));
+							optarg, strerror(errno));
 					exit(1);
 				}
 				break;
 
 			/* error handler */
 			case 'E':
-				if( (strlen(optarg) == 0) || (optarg[0] != '/') )
+				if ((strlen(optarg) == 0) || (optarg[0] != '/'))
 				{
 					fprintf(stderr, "Error: Invalid error handler: %s\n",
-						optarg);
+							optarg);
 					exit(1);
 				}
 				conf.error_handler = optarg;
@@ -861,10 +871,10 @@ int main (int argc, char **argv)
 
 			/* index file */
 			case 'I':
-				if( (strlen(optarg) == 0) || (optarg[0] == '/') )
+				if ((strlen(optarg) == 0) || (optarg[0] == '/'))
 				{
 					fprintf(stderr, "Error: Invalid index page: %s\n",
-						optarg);
+							optarg);
 					exit(1);
 				}
 				conf.index_file = optarg;
@@ -892,7 +902,7 @@ int main (int argc, char **argv)
 
 			/* interpreter */
 			case 'i':
-				if( (optarg[0] == '.') && (port = strchr(optarg, '=')) )
+				if ((optarg[0] == '.') && (port = strchr(optarg, '=')))
 				{
 					*port++ = 0;
 					uh_interpreter_add(optarg, port);
@@ -900,7 +910,7 @@ int main (int argc, char **argv)
 				else
 				{
 					fprintf(stderr, "Error: Invalid interpreter: %s\n",
-						optarg);
+							optarg);
 					exit(1);
 				}
 				break;
@@ -942,7 +952,7 @@ int main (int argc, char **argv)
 
 			/* urldecode */
 			case 'd':
-				if( (port = malloc(strlen(optarg)+1)) != NULL )
+				if ((port = malloc(strlen(optarg)+1)) != NULL)
 				{
 					/* "decode" plus to space to retain compat */
 					for (opt = 0; optarg[opt]; opt++)
@@ -951,7 +961,7 @@ int main (int argc, char **argv)
 					/* opt now contains strlen(optarg) -- no need to re-scan */
 					memset(port, 0, opt+1);
 					if (uh_urldecode(port, opt, optarg, opt) < 0)
-					    fprintf( stderr, "uhttpd: invalid encoding\n" );
+					    fprintf(stderr, "uhttpd: invalid encoding\n");
 
 					printf("%s", port);
 					free(port);
@@ -1015,21 +1025,21 @@ int main (int argc, char **argv)
 	}
 
 #ifdef HAVE_TLS
-	if( (tls == 1) && (keys < 2) )
+	if ((tls == 1) && (keys < 2))
 	{
 		fprintf(stderr, "Error: Missing private key or certificate file\n");
 		exit(1);
 	}
 #endif
 
-	if( bound < 1 )
+	if (bound < 1)
 	{
 		fprintf(stderr, "Error: No sockets bound, unable to continue\n");
 		exit(1);
 	}
 
 	/* default docroot */
-	if( !conf.docroot[0] && !realpath(".", conf.docroot) )
+	if (!conf.docroot[0] && !realpath(".", conf.docroot))
 	{
 		fprintf(stderr, "Error: Can not determine default document root: %s\n",
 			strerror(errno));
@@ -1037,56 +1047,55 @@ int main (int argc, char **argv)
 	}
 
 	/* default realm */
-	if( ! conf.realm )
+	if (!conf.realm)
 		conf.realm = "Protected Area";
 
 	/* config file */
 	uh_config_parse(&conf);
 
 	/* default network timeout */
-	if( conf.network_timeout <= 0 )
+	if (conf.network_timeout <= 0)
 		conf.network_timeout = 30;
 
 #if defined(HAVE_CGI) || defined(HAVE_LUA)
 	/* default script timeout */
-	if( conf.script_timeout <= 0 )
+	if (conf.script_timeout <= 0)
 		conf.script_timeout = 60;
 #endif
 
 #ifdef HAVE_CGI
 	/* default cgi prefix */
-	if( ! conf.cgi_prefix )
+	if (!conf.cgi_prefix)
 		conf.cgi_prefix = "/cgi-bin";
 #endif
 
 #ifdef HAVE_LUA
 	/* load Lua plugin */
-	if( ! (lib = dlopen("uhttpd_lua.so", RTLD_LAZY | RTLD_GLOBAL)) )
+	if (!(lib = dlopen("uhttpd_lua.so", RTLD_LAZY | RTLD_GLOBAL)))
 	{
 		fprintf(stderr,
-			"Notice: Unable to load Lua plugin - disabling Lua support! "
-			"(Reason: %s)\n", dlerror()
-		);
+				"Notice: Unable to load Lua plugin - disabling Lua support! "
+				"(Reason: %s)\n", dlerror());
 	}
 	else
 	{
 		/* resolve functions */
-		if( !(conf.lua_init    = dlsym(lib, "uh_lua_init"))    ||
+		if (!(conf.lua_init    = dlsym(lib, "uh_lua_init"))    ||
 		    !(conf.lua_close   = dlsym(lib, "uh_lua_close"))   ||
-		    !(conf.lua_request = dlsym(lib, "uh_lua_request"))
-		) {
+		    !(conf.lua_request = dlsym(lib, "uh_lua_request")))
+		{
 			fprintf(stderr,
-				"Error: Failed to lookup required symbols "
-				"in Lua plugin: %s\n", dlerror()
+					"Error: Failed to lookup required symbols "
+					"in Lua plugin: %s\n", dlerror()
 			);
 			exit(1);
 		}
 
 		/* init Lua runtime if handler is specified */
-		if( conf.lua_handler )
+		if (conf.lua_handler)
 		{
 			/* default lua prefix */
-			if( ! conf.lua_prefix )
+			if (!conf.lua_prefix)
 				conf.lua_prefix = "/lua";
 
 			conf.lua_state = conf.lua_init(&conf);
@@ -1095,9 +1104,9 @@ int main (int argc, char **argv)
 #endif
 
 	/* fork (if not disabled) */
-	if( ! nofork )
+	if (!nofork)
 	{
-		switch( fork() )
+		switch (fork())
 		{
 			case -1:
 				perror("fork()");
@@ -1105,16 +1114,16 @@ int main (int argc, char **argv)
 
 			case 0:
 				/* daemon setup */
-				if( chdir("/") )
+				if (chdir("/"))
 					perror("chdir()");
 
-				if( (cur_fd = open("/dev/null", O_WRONLY)) > -1 )
+				if ((cur_fd = open("/dev/null", O_WRONLY)) > -1)
 					dup2(cur_fd, 0);
 
-				if( (cur_fd = open("/dev/null", O_RDONLY)) > -1 )
+				if ((cur_fd = open("/dev/null", O_RDONLY)) > -1)
 					dup2(cur_fd, 1);
 
-				if( (cur_fd = open("/dev/null", O_RDONLY)) > -1 )
+				if ((cur_fd = open("/dev/null", O_RDONLY)) > -1)
 					dup2(cur_fd, 2);
 
 				break;
@@ -1129,7 +1138,7 @@ int main (int argc, char **argv)
 
 #ifdef HAVE_LUA
 	/* destroy the Lua state */
-	if( conf.lua_state != NULL )
+	if (conf.lua_state != NULL)
 		conf.lua_close(conf.lua_state);
 #endif