uhttpd: display errors in init script, code formatting changes, bump package version
authorJo-Philipp Wich <jow@openwrt.org>
Thu, 3 May 2012 17:19:22 +0000 (17:19 +0000)
committerJo-Philipp Wich <jow@openwrt.org>
Thu, 3 May 2012 17:19:22 +0000 (17:19 +0000)
SVN-Revision: 31572

package/uhttpd/Makefile
package/uhttpd/files/uhttpd.init
package/uhttpd/src/uhttpd-cgi.c
package/uhttpd/src/uhttpd-file.c
package/uhttpd/src/uhttpd-lua.c
package/uhttpd/src/uhttpd-utils.c
package/uhttpd/src/uhttpd.c

index 5688073adda7901ebb306fd47e8ef3c378dfe54b..0331470bf372ac204572fa3b2b2e46f0ffa70e9b 100644 (file)
@@ -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 := \
index c370f901099b6456ede8d2acb8abbd0e1d40c498..d4037a1b981b0800a3047fe948d7b14ff0ff1e3f 100755 (executable)
@@ -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()
index 16bfda7dba5b6400bb5f0af91aee2adc3951c503..f852125698b15502f5c132a01216a0b43f809eb0 100644 (file)
@@ -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;
        }
 }
-
index 816a69124ed47d1e214152aedfff5ac05a777fb5..0d9a207b119c9b46c93c3393f6bbf72f1782f53b 100644 (file)
@@ -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);
 }
-
index 83b0b0a26b9cfbced8afb9c5289948362f59fd9d..ea6f26cc9bbf50ed3c0cce4d22cff0041f11e670 100644 (file)
@@ -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);
index 1dac33db0ff6973fc421997b1afd8ca26a8dd20f..18969e74dd80a519197cf02c91a5940cc03e6a5a 100644 (file)
@@ -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));
 
-                       ifno_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 */
-       ifstrncmp(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;
        }
 
index 78e0b5d952ec8027b4c38f0dc89649640c2b8f5a..05928113862f26b722306dc3421ef18d9c8c63fd 100644 (file)
@@ -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;
 
-                       ifsetsockopt(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 ifhdrname && !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;
 
-       ifuh_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? */
-                                               ifconf->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? */
-                                               ifconf->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