]> sourceware.org Git - lvm2.git/commit
dmsetup: fix error propagation in _display_info_cols()
authorBryn M. Reeves <bmr@redhat.com>
Thu, 28 Jun 2018 13:25:30 +0000 (14:25 +0100)
committerMarian Csontos <mcsontos@redhat.com>
Wed, 25 Jul 2018 08:55:28 +0000 (10:55 +0200)
commit951676a59eb2b3130abb9eec690206665708b0d0
tree7737230285681a15f15a901d231695bf36e23e80
parent4456d9aa77de7ac035ab975d87b0696e33370ab4
dmsetup: fix error propagation in _display_info_cols()

Commit 3f35146 added a check on the value returned by the
_display_info_cols() function:

  1024         if (!_switches[COLS_ARG])
  1025                 _display_info_long(dmt, &info);
  1026         else
  1027                 r = _display_info_cols(dmt, &info);
  1028
  1029         return r;

This exposes a bug in the dmstats code in _display_info_cols:
the fact that a device has no regions is explicitly not an error
(and is documented as such in the code), but since the return
code is not changed before leaving the function it is now treated
as an error leading to:

  # dmstats list
  Command failed.

When no regions exist.

Set the return code to the correct value before returning.

(cherry picked from commit 29b9ccd261be025aaf75e58e5d2547e818ef22c3)
tools/dmsetup.c
This page took 0.033429 seconds and 5 git commands to generate.