Changed the len type to ssize_t because we need to detect errors in the

recv() call.  Thanks to Tom Cross for discovering this bug.

2 files changed, 6 insertions, 2 deletions

diff --git a/ChangeLog b/ChangeLog
index f482c5a..7222150 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2002-04-26  Robert James Kaes  <rjkaes@flarenet.com>
 
+	* src/reqs.c (connect_to_tunnel): Changed the len type to an
+	signed integer so that we can detect error conditions.  Thanks to
+	Tom Cross for pointing out this problem.
+
 	* src/log.c (send_stored_logs): Check the return value of
 	hashmap_first() since it could be -1, indicating an empty hashmap.
 
diff --git a/src/reqs.c b/src/reqs.c
index f5cf0a6..73a581a 100644
--- a/src/reqs.c
+++ b/src/reqs.c
@@ -1,4 +1,4 @@
-/* $Id: reqs.c,v 1.68 2002-04-26 16:43:20 rjkaes Exp $
+/* $Id: reqs.c,v 1.69 2002-04-26 19:33:09 rjkaes Exp $
  *
  * This is where all the work in tinyproxy is actually done. Incoming
  * connections have a new thread created for them. The thread then
@@ -1077,7 +1077,7 @@ static int
 connect_to_tunnel(struct conn_s *connptr)
 {
 	char *request_buf;
-	size_t len;
+	ssize_t len;
 	int pos;
 
 	request_buf = safemalloc(HTTP_LINE_LENGTH);