don't mmap

Before we mmap(2) file for reading, and use a buffer to handle CGI
scripts.  Turns out, for sequential access over the whole mmap isn't
better than our loop on read.  This has also the additional advantage
that we can use handle_cgi (now handle_copy) for both files and CGI,
which is pretty cool.

This also fixes a nasty bug where we could hang a connection forever,
because we scheduled the wrong type of event (read on POLLOUT and
write on POLLIN, it's the other way around!)

1 files changed, 2 insertions, 3 deletions

diff --git a/gmid.h b/gmid.h
index 2fdfc7e..6d339c1 100644
--- a/gmid.h
+++ b/gmid.h
@@ -171,9 +171,8 @@ struct client {
 	const char	*meta;
 	int		 fd, pfd;
 	DIR		*dir;
-	char		 sbuf[1024];	  /* static buffer */
-	void		*buf, *i;	  /* mmap buffer */
-	ssize_t		 len, off;	  /* mmap/static buffer  */
+	char		 sbuf[BUFSIZ];
+	ssize_t		 len, off;
 	struct sockaddr_storage	 addr;
 	struct vhost	*host;	/* host she's talking to */
 };