diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2016-03-01 13:58:15 +0100 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2016-03-01 14:02:43 +0100 |
| commit | 36d0dcef38ec3ce89f3c13be272efc69f981b207 (patch) | |
| tree | 5753ba7afc4bd797618d42bac89c37867c3e86f9 | |
| parent | 8173c2ff9af805a1f8eb79fb4bd223622f79ad0d (diff) | |
| download | lvm2-36d0dcef38ec3ce89f3c13be272efc69f981b207.tar.gz | |
coverity: eliminate DEADCODE
Coverity notice this cannot be NULL: cur = &dms->regions[*cur_r]
so avoid NULL checking and simplify form.
| -rw-r--r-- | libdm/libdm-stats.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/libdm/libdm-stats.c b/libdm/libdm-stats.c index 82d3fa5d0..1dce3ebdb 100644 --- a/libdm/libdm-stats.c +++ b/libdm/libdm-stats.c @@ -957,15 +957,13 @@ bad: static void _stats_walk_next(const struct dm_stats *dms, int region, uint64_t *cur_r, uint64_t *cur_a) { - struct dm_stats_region *cur = NULL; + struct dm_stats_region *cur; int present; if (!dms || !dms->regions) return; - if (!(cur = &dms->regions[*cur_r])) - return; - + cur = dms->regions + *cur_r; present = _stats_region_present(cur); if (region && present) |