clang: ensure extents is usable
authorZdenek Kabelac <zkabelac@redhat.com>
Sun, 16 Feb 2025 20:43:23 +0000 (21:43 +0100)
committerZdenek Kabelac <zkabelac@redhat.com>
Mon, 17 Feb 2025 14:49:54 +0000 (15:49 +0100)
Although the code was exiting only for  (update == 0),
the later code actually used requires 'extents' to exist
also for  (update != 0).

TODO: The logic here is not very clear, more testing needed.

libdm/libdm-stats.c

index fa6d6d241a25e02004d2f0209d77778fd86eb508..c869dc9b6b47e5122eed983ab217a560fd3b6a12 100644 (file)
@@ -4753,8 +4753,7 @@ static uint64_t *_stats_map_file_regions(struct dm_stats *dms, int fd,
 
        if (!(extents = _stats_get_extents_for_file(extent_mem, fd, count))) {
                log_very_verbose("No extents found in fd %d", fd);
-               if (!update)
-                       goto out;
+               goto out;
        }
 
        if (update) {
This page took 0.070804 seconds and 5 git commands to generate.