aboutsummaryrefslogtreecommitdiffstats
path: root/tap-httpstat.c
diff options
context:
space:
mode:
authorRonnie Sahlberg <ronnie_sahlberg@ozemail.com.au>2006-05-01 07:43:47 +0000
committerRonnie Sahlberg <ronnie_sahlberg@ozemail.com.au>2006-05-01 07:43:47 +0000
commitc978ebe47c764947e945d2a17bccce2ec79c6a1a (patch)
tree64296d61f4ba71d64e34794c27f421ce0088068f /tap-httpstat.c
parent9c18c35c38b0e91758f5b329da25d2e34cac0422 (diff)
coverity 155
coverity showed this as a null dereference which was a false positive. sp can never be null here. there is a bug however since with this invalid test the memory release functions will never be called and we thus had a memory leak here. svn path=/trunk/; revision=18053
Diffstat (limited to 'tap-httpstat.c')
-rw-r--r--tap-httpstat.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/tap-httpstat.c b/tap-httpstat.c
index 0e55e5e5a7..352485d205 100644
--- a/tap-httpstat.c
+++ b/tap-httpstat.c
@@ -181,10 +181,10 @@ static void
httpstat_reset(void *psp )
{
httpstat_t *sp=psp;
- if (!sp) {
- g_hash_table_foreach( sp->hash_responses, (GHFunc)http_reset_hash_responses, NULL);
- g_hash_table_foreach( sp->hash_requests, (GHFunc)http_reset_hash_requests, NULL);
- }
+
+ g_hash_table_foreach( sp->hash_responses, (GHFunc)http_reset_hash_responses, NULL);
+ g_hash_table_foreach( sp->hash_requests, (GHFunc)http_reset_hash_requests, NULL);
+
}
static int