diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2020-12-07 16:16:55 +0100 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2020-12-08 20:32:34 +0100 |
| commit | 47608ff49b874f2a34b9d8644f53028d5ffa1fd3 (patch) | |
| tree | 3ca6435f6537bf415c4c4af5bad528046580a19c /scripts | |
| parent | 7691213a91ca7cc2d5518db31d04c8afa74622e7 (diff) | |
| download | lvm2-47608ff49b874f2a34b9d8644f53028d5ffa1fd3.tar.gz | |
fsadm: fix unbound variable usage
When 'fsadm resize vg/lv' is used without size, it should just
resize filesystem to match device - but since we now check
for unbound variable in bash - the previous usage no longer
works and needs explicit check.
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/fsadm.sh | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/scripts/fsadm.sh b/scripts/fsadm.sh index 9b95d318d..2cb1fc75b 100755 --- a/scripts/fsadm.sh +++ b/scripts/fsadm.sh @@ -798,6 +798,7 @@ fi CHECK="" RESIZE="" +NEWSIZE="" while [ "$#" -ne 0 ] do @@ -811,8 +812,11 @@ do "-y"|"--yes") YES="-y" ;; "-l"|"--lvresize") DO_LVRESIZE=1 ;; "-c"|"--cryptresize") DO_CRYPTRESIZE=1 ;; - "check") CHECK=$2 ; shift ;; - "resize") RESIZE=$2 ; NEWSIZE=$3 ; shift 2 ;; + "check") test -z "${2-}" && error "Missing <device>. (see: $TOOL --help)" + CHECK=$2 ; shift ;; + "resize") test -z "${2-}" && error "Missing <device>. (see: $TOOL --help)" + RESIZE=$2 ; shift + if test -n "${2-}" ; then NEWSIZE="${2-}" ; shift ; fi ;; *) error "Wrong argument \"$1\". (see: $TOOL --help)" esac shift |