diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2021-09-18 21:01:48 +0200 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2021-09-20 13:58:57 +0200 |
| commit | 93d565eda992abc0460cc8fc7b9998bd1996a447 (patch) | |
| tree | 75a7d267d8484f77350cc642da9e232666e786b7 /tools/vgimportclone.c | |
| parent | 5126ac7c3aa4f9f47a36c66095246d5b13871e61 (diff) | |
| download | lvm2-93d565eda992abc0460cc8fc7b9998bd1996a447.tar.gz | |
gcc-fanalyzer: use zeroed memory
Analyzer here was rather confused about possiblity of loosing previously
assigned device pointers - fixed by passing zero initialize memory
before first assign.
Diffstat (limited to 'tools/vgimportclone.c')
| -rw-r--r-- | tools/vgimportclone.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/vgimportclone.c b/tools/vgimportclone.c index 8aa0dbba9..23bb6271f 100644 --- a/tools/vgimportclone.c +++ b/tools/vgimportclone.c @@ -194,7 +194,7 @@ static int _get_other_devs(struct cmd_context *cmd, struct dm_list *new_devs, st while ((dev = dev_iter_get(cmd, iter))) { if (_get_device_list(new_devs, dev)) continue; - if (!(devl = malloc(sizeof(*devl)))) { + if (!(devl = zalloc(sizeof(*devl)))) { r = 0; goto_bad; } @@ -278,7 +278,7 @@ int vgimportclone(struct cmd_context *cmd, int argc, char **argv) goto out; } - if (!(devl = malloc(sizeof(*devl)))) + if (!(devl = zalloc(sizeof(*devl)))) goto_out; devl->dev = dev; |