led_ctrl: fix permission settings for test directories
authorSimon Guinot <simon@sequanux.org>
Tue, 9 Nov 2010 15:14:17 +0000 (16:14 +0100)
committerSimon Guinot <simon@sequanux.org>
Tue, 9 Nov 2010 15:14:17 +0000 (16:14 +0100)
  find $SYSPATH -type d -exec chmod 500 '{}'"

is not portable and return the following error under ash shell:

  find: -exec CMD must end by ';'

This patch replace the find and execute command with the more portable
and simple chmod calls.

led_ctrl/led_ctrl

index 762ba27..db43379 100755 (executable)
@@ -44,6 +44,7 @@ set_led_attr() # <led> <attribute> <value>
 
        IFS_ORIG="$IFS"
        IFS=$'\n'
+
        for attr in $(ls -1 $SYSPATH/$1/$2); do
                log_and_exec "echo $3 > \"$attr\"" || { err=1; break; }
        done
@@ -99,7 +100,8 @@ run_test_suite()
        touch $SYSPATH/led\ 3/attr1
        mkdir -p $SYSPATH/led\ 4
        touch $SYSPATH/led\ 4/attr1
-       find $SYSPATH -type d -exec chmod 500 '{}' +
+       chmod 500 $SYSPATH/*
+       chmod 500 $SYSPATH
 
        CONFILE=$TEST_DIR/conf
        cat > $CONFILE <<- EOF
@@ -154,7 +156,7 @@ run_test_suite()
        run_test 'commented-out led mode' \
                'set_led_mode mode6 2>&1 | grep -q "no mode"'
 
-       chmod -R u+w $SYSPATH
+       chmod -R 700 $SYSPATH
        rm -fr $TEST_DIR
 
        return $TEST_ERR