diff options
| author | sahlberg <sahlberg@f5534014-38df-0310-8fa8-9805f1628bb7> | 2006-05-01 07:43:47 +0000 |
|---|---|---|
| committer | sahlberg <sahlberg@f5534014-38df-0310-8fa8-9805f1628bb7> | 2006-05-01 07:43:47 +0000 |
| commit | 1823b4b6826bbc46bb530717dd7b175e3fdf54f1 (patch) | |
| tree | 64296d61f4ba71d64e34794c27f421ce0088068f /tap-httpstat.c | |
| parent | 623d58fc9151758d0fc8456aaeb40b7465e9ce77 (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.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@18053 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'tap-httpstat.c')
| -rw-r--r-- | tap-httpstat.c | 8 |
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 |