bughz/tinyproxy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

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

Browse Source 
recv() call.  Thanks to Tom Cross for discovering this bug.
This commit is contained in:
Robert James Kaes 2002-04-26 19:33:09 +00:00
parent 9554989110
commit 1a8c914dd2
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ChangeLog
View File

@ -1,5 +1,9 @@
2002-04-26 Robert James Kaes <rjkaes@flarenet.com> 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 * src/log.c (send_stored_logs): Check the return value of
hashmap_first() since it could be -1, indicating an empty hashmap. hashmap_first() since it could be -1, indicating an empty hashmap.

4
src/reqs.c
View File

@ -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 * This is where all the work in tinyproxy is actually done. Incoming
* connections have a new thread created for them. The thread then * connections have a new thread created for them. The thread then
@ -1077,7 +1077,7 @@ static int
connect_to_tunnel(struct conn_s *connptr) connect_to_tunnel(struct conn_s *connptr)
{ {
char *request_buf; char *request_buf;
size_t len; ssize_t len;
int pos; int pos;
request_buf = safemalloc(HTTP_LINE_LENGTH); request_buf = safemalloc(HTTP_LINE_LENGTH);