diff options
| author | Bryn M. Reeves <bmr@redhat.com> | 2015-08-10 10:01:18 +0100 |
|---|---|---|
| committer | Bryn M. Reeves <bmr@redhat.com> | 2015-08-10 20:12:53 +0100 |
| commit | 3b74824985f7e4e90f0ba9668724abd08d5fef74 (patch) | |
| tree | 363628c972d58a7033360b171d180face7608001 | |
| parent | 6bd5bf3cb5a1c01ec8be8bb714c7c0461d6bec58 (diff) | |
| download | lvm2-3b74824985f7e4e90f0ba9668724abd08d5fef74.tar.gz | |
dmsetup: remove bogus !_report test in _stats_report (Coverity)
There's no point testing _report here in _stats_report: it's always
initialised before the function is called and if the check did fail
we'd end up freeing an uninitialized dm_task in the error path.
tools/dmsetup.c: 4389 in _stats_report() - Declaring variable "dmt" without initializer.
| -rw-r--r-- | tools/dmsetup.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/tools/dmsetup.c b/tools/dmsetup.c index de67b0341..2ee432b3a 100644 --- a/tools/dmsetup.c +++ b/tools/dmsetup.c @@ -4397,9 +4397,6 @@ static int _stats_report(CMD_ARGS) name = argv[1]; } - if (!_report) - goto out; - if (!(dmt = dm_task_create(DM_DEVICE_INFO))) return 0; |