From 1a8c914dd2872ea29bd656d54a63e5ddab2c082b Mon Sep 17 00:00:00 2001 From: Robert James Kaes Date: Fri, 26 Apr 2002 19:33:09 +0000 Subject: 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. --- ChangeLog | 4 ++++ src/reqs.c | 4 ++-- 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 + * 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); -- cgit v1.2.3