Fix hijack.sh sudo() to use an array, support whitespace and behave

Overriden commands should work exactly like real commands and support
whitespace and special characters exactly like the real command.

I usually disable the sudo() override locally when using `set -x`
to reduce trace noise considerably. The previous difference between the
sudo() override and the built-in sudo() cost me couple hours twice:

- WIP #897 (comment)
- Merged commit 232d40a

No more!

Also fix the return value: 2 means SKIP.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>

Author: marc-hb

case-lib/hijack.sh

@@ -177,20 +177,25 @@ SUDO_LEVEL=""
 # overwrite the sudo command, sudo in the script can direct using sudo command
 sudo()
 {
+    local cmd=( "$@" )
+
     func_hijack_setup_sudo_level || true
     local cmd
     case $SUDO_LEVEL in
-        '0')    cmd="$*" # as root
+        '0')    "${cmd[@]}" # as root
+                return $?
         ;;
-        '1')    cmd="$SUDO_CMD env 'PATH=$PATH' $*" # sudo without passwd
+        '1')    "$SUDO_CMD" env PATH="$PATH" "${cmd[@]}" # sudo without passwd
+                return $?
         ;;
-        '2')    cmd="echo '$SUDO_PASSWD' | $SUDO_CMD -S env 'PATH=$PATH' $*" # sudo need passwd
+        # sudo need passwd
+        '2')    echo "$SUDO_PASSWD" | "$SUDO_CMD" --stdin env PATH="$PATH" "${cmd[@]}"
+                return $?
         ;;
         *)      # without sudo permission
             dlogw "Need root privilege to run $*"
-            return 2
+            return 1
     esac
-    eval "$cmd"
 }
 
 func_hijack_setup_sudo_level()

test-case/check-ipc-flood.sh

@@ -50,7 +50,7 @@ do
     setup_kernel_check_point
     dlogi "===== [$i/$loop_cnt] loop Begin ====="
     dlogc "sudo bash -c 'echo $lpc_loop_cnt > $ipc_flood_dfs'"
-    sudo bash -c "'echo $lpc_loop_cnt > $ipc_flood_dfs'"
+    sudo bash -c "echo $lpc_loop_cnt > $ipc_flood_dfs"
 
     # check kernel log for each iteration to catch issues
     sof-kernel-log-check.sh "$KERNEL_CHECKPOINT" || die "Caught error in kernel log"