diff options
author | Anoop C S <anoopcs@redhat.com> | 2017-10-11 13:29:13 +0530 |
---|---|---|
committer | Jeff Darcy <jeff@pl.atyp.us> | 2017-10-18 14:27:41 +0000 |
commit | 087703fae449b725ebd82bc3cd65763a016fb098 (patch) | |
tree | 34ba22f3c5547b0b6e8a29a6b10c719639ebd005 /extras/hook-scripts/set/post/S30samba-set.sh | |
parent | f0650fef60ca557ea1c60458f705ac9b52854b39 (diff) |
extras/hooks: Fix errors reported via shellcheck utility
Change-Id: I217c6b2a39955f1709bb3452b00d33c2dcb60faa
BUG: 1500649
Signed-off-by: Anoop C S <anoopcs@redhat.com>
Diffstat (limited to 'extras/hook-scripts/set/post/S30samba-set.sh')
-rwxr-xr-x | extras/hook-scripts/set/post/S30samba-set.sh | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/extras/hook-scripts/set/post/S30samba-set.sh b/extras/hook-scripts/set/post/S30samba-set.sh index 97d067fc33f..c21cfb57205 100755 --- a/extras/hook-scripts/set/post/S30samba-set.sh +++ b/extras/hook-scripts/set/post/S30samba-set.sh @@ -28,7 +28,7 @@ USERSMB_SET="" USERCIFS_SET="" function parse_args () { - ARGS=$(getopt -l $OPTSPEC --name $PROGNAME -o "o:" -- $@) + ARGS=$(getopt -l $OPTSPEC --name $PROGNAME -o "o:" -- "$@") eval set -- "$ARGS" while true; do @@ -123,23 +123,23 @@ function get_smb () { usersmbvalue=$(grep user.smb $GLUSTERD_WORKDIR/vols/"$volname"/info |\ cut -d"=" -f2) - if [[ $usercifsvalue = "disable" || $usersmbvalue = "disable" ]]; then + if [ $usercifsvalue = "disable" ] || [ $usersmbvalue = "disable" ]; then uservalue="disable" fi echo "$uservalue" } -parse_args $@ -if [ "0" = $(is_volume_started "$VOL") ]; then +parse_args "$@" +if [ "0" = "$(is_volume_started "$VOL")" ]; then exit 0 fi -if [[ "$USERCIFS_SET" = "YES" || "$USERSMB_SET" = "YES" ]]; then +if [ "$USERCIFS_SET" = "YES" ] || [ "$USERSMB_SET" = "YES" ]; then #Find smb.conf, smbd pid directory and smbd logfile path find_config_info - if [ $(get_smb "$VOL") = "disable" ]; then + if [ "$(get_smb "$VOL")" = "disable" ]; then del_samba_share $VOL sighup_samba else |