diff options
author | Vincent Sanders <vince@kyllikki.org> | 2022-09-11 13:50:55 +0100 |
---|---|---|
committer | Vincent Sanders <vince@kyllikki.org> | 2022-09-11 13:50:55 +0100 |
commit | 40acd620b50f9cc81305c564cee45cbb3de880f9 (patch) | |
tree | 1a42c7039c0bba1706ca0d235fa6c317c7fa5f22 /content/handlers/image | |
parent | c6eea438ac6698f60bc4c7f1d490db1c599f5488 (diff) | |
download | netsurf-40acd620b50f9cc81305c564cee45cbb3de880f9.tar.gz netsurf-40acd620b50f9cc81305c564cee45cbb3de880f9.tar.bz2 |
Ensure image cache logging does not cause division by zero at exit
Diffstat (limited to 'content/handlers/image')
-rw-r--r-- | content/handlers/image/image_cache.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/content/handlers/image/image_cache.c b/content/handlers/image/image_cache.c index 6ce6b5234..ae53192a9 100644 --- a/content/handlers/image/image_cache.c +++ b/content/handlers/image/image_cache.c @@ -438,6 +438,7 @@ image_cache_init(const struct image_cache_parameters *image_cache_parameters) nserror image_cache_fini(void) { unsigned int op_count; + uint64_t op_size; guit->misc->schedule(-1, image_cache__background_update, image_cache); @@ -452,6 +453,10 @@ nserror image_cache_fini(void) image_cache->miss_count + image_cache->fail_count; + op_size = image_cache->hit_size + + image_cache->miss_size + + image_cache->fail_size; + NSLOG(netsurf, INFO, "Age %ds", image_cache->current_age / 1000); NSLOG(netsurf, INFO, "Peak size %"PRIsizet" (in %d)", image_cache->max_bitmap_size, @@ -460,12 +465,7 @@ nserror image_cache_fini(void) image_cache->max_bitmap_count, image_cache->max_bitmap_count_size); - if (op_count > 0) { - uint64_t op_size; - - op_size = image_cache->hit_size + - image_cache->miss_size + - image_cache->fail_size; + if ((op_count > 0) && (op_size >0)) { NSLOG(netsurf, INFO, "Cache total/hit/miss/fail (counts) %d/%d/%d/%d (100%%/%d%%/%d%%/%d%%)", |