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

Check the return value of hashmap_first()

Browse Source
This commit is contained in:
Robert James Kaes 2002-04-26 16:50:55 +00:00
parent 0e7182533c
commit e9f5b2e75f
2 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ChangeLog
View File

@ -1,5 +1,8 @@
2002-04-26 Robert James Kaes <rjkaes@flarenet.com> 2002-04-26 Robert James Kaes <rjkaes@flarenet.com>
* src/log.c (send_stored_logs): Check the return value of
hashmap_first() since it could be -1, indicating an empty hashmap.
* src/reqs.c (process_client_headers): * src/reqs.c (process_client_headers):
(process_server_headers): Test the return value of hashmap_first() (process_server_headers): Test the return value of hashmap_first()
since the hashmap could be empty (returning a -1 via hashmap_first.) since the hashmap could be empty (returning a -1 via hashmap_first.)

17
src/log.c
View File

@ -1,4 +1,4 @@
/* $Id: log.c,v 1.18 2002-04-25 18:56:43 rjkaes Exp $ /* $Id: log.c,v 1.19 2002-04-26 16:50:55 rjkaes Exp $
* *
* Logs the various messages which tinyproxy produces to either a log file or * Logs the various messages which tinyproxy produces to either a log file or
* the syslog daemon. Not much to it... * the syslog daemon. Not much to it...
@ -159,13 +159,14 @@ send_stored_logs(void)
char *level; char *level;
char *string; char *string;
for (iter = hashmap_first(log_message_storage); iter = hashmap_first(log_message_storage);
iter != hashmap_is_end(log_message_storage, iter); if (iter >= 0) {
++iter) { for ( ; iter != hashmap_is_end(log_message_storage, iter); ++iter) {
hashmap_return_entry(log_message_storage, hashmap_return_entry(log_message_storage,
iter, iter,
&string, &string,
(void **)&level); (void **)&level);
}
} }
hashmap_delete(log_message_storage); hashmap_delete(log_message_storage);