nsswitch: Fix shellcheck errors in test_rfc2307_mapping.sh
authorAndreas Schneider <asn@samba.org>
Tue, 22 Feb 2022 16:58:47 +0000 (17:58 +0100)
committerPavel Filipensky <pfilipensky@samba.org>
Wed, 17 Aug 2022 10:08:35 +0000 (10:08 +0000)
nsswitch/tests/test_rfc2307_mapping.sh:65:139: error: Double quote array
expansions to avoid re-splitting elements. [SC2068]

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Pavel Filipenský <pfilipensky@samba.org>
nsswitch/tests/test_rfc2307_mapping.sh

index 8cd2e5dc57e53e958ce8d09593ba6b9d4928a272..d6d39a10b689bfe3c797eb8a8a598a389b73a599 100755 (executable)
@@ -63,10 +63,12 @@ knownfail()
 }
 
 # Create new testing account
-testit "user add" $PYTHON $samba_tool user create --given-name="rfc2307" --surname="Tester" --initial="UT" rfc2307_test_user testp@ssw0Rd $@
+testit "user add" $PYTHON $samba_tool user create --given-name="rfc2307" \
+       --surname="Tester" --initial="UT" rfc2307_test_user testp@ssw0Rd "$@"
 
 #test creation of six different groups
-testit "group add" $PYTHON $samba_tool group add $CONFIG --group-scope='Domain' --group-type='Security' rfc2307_test_group $@
+testit "group add" $PYTHON $samba_tool group add $CONFIG \
+       --group-scope='Domain' --group-type='Security' rfc2307_test_group "$@"
 
 # Create new testing group
 
@@ -115,7 +117,9 @@ add: uidNumber
 uidNumber: $UID_RFC2307TEST
 EOF
 
-testit "modify gidNumber on group" $VALGRIND $ldbmodify -H ldap://$SERVER $PREFIX/tmpldbmodify -U$DOMAIN/$USERNAME%$PASSWORD $@ || failed=$(expr $failed + 1)
+testit "modify gidNumber on group" $VALGRIND $ldbmodify -H ldap://$SERVER \
+       $PREFIX/tmpldbmodify -U$DOMAIN/$USERNAME%$PASSWORD "$@" ||
+       failed=$(expr $failed + 1)
 
 # Then add a gidNumber to the group record using ldbmodify
 cat >$PREFIX/tmpldbmodify <<EOF
@@ -125,7 +129,9 @@ add: gidNumber
 gidNumber: $GID_RFC2307TEST
 EOF
 
-testit "modify gidNumber on group" $VALGRIND $ldbmodify -H ldap://$SERVER $PREFIX/tmpldbmodify -U$DOMAIN/$USERNAME%$PASSWORD $@ || failed=$(expr $failed + 1)
+testit "modify gidNumber on group" $VALGRIND $ldbmodify -H ldap://$SERVER \
+       $PREFIX/tmpldbmodify -U$DOMAIN/$USERNAME%$PASSWORD "$@" ||
+       failed=$(expr $failed + 1)
 
 rm -f $PREFIX/tmpldbmodify
 
@@ -181,7 +187,7 @@ else
        echo "success: wbinfo -Y check for sane mapping"
 fi
 
-testit "group delete" $PYTHON $samba_tool group delete rfc2307_test_group $@
-testit "user delete" $PYTHON $samba_tool user delete rfc2307_test_user $@
+testit "group delete" $PYTHON $samba_tool group delete rfc2307_test_group "$@"
+testit "user delete" $PYTHON $samba_tool user delete rfc2307_test_user "$@"
 
 exit $failed