From a0f779c5fa6e8706274fb3072b2a3d815061bcf7 Mon Sep 17 00:00:00 2001 From: Srikanth Muppandam Date: Tue, 7 Apr 2026 06:49:50 +0530 Subject: [PATCH] Replace the non-standard function syntax in pidkiller(), remove the non-POSIX disown usage, and quote variables passed to test and kill. These changes address ShellCheck warnings without changing the current runtime flow or behavior. Signed-off-by: Srikanth Muppandam --- Runner/utils/platform.sh | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/Runner/utils/platform.sh b/Runner/utils/platform.sh index fcd74595..348fd01a 100755 --- a/Runner/utils/platform.sh +++ b/Runner/utils/platform.sh @@ -6,19 +6,14 @@ # Detect Android userland ANDROID_PATH=/system/build.prop if [ -f $ANDROID_PATH ]; then - ANDROID=1 # shellcheck disable=SC2209,SC2034 SHELL_CMD=sh else - ANDROID=0 # shellcheck disable=SC2209,SC2034 SHELL_CMD=bash fi -function pidkiller() +pidkiller() { - if [ $ANDROID -eq 0 ]; then - disown $1 - fi - kill -9 $1 >/dev/null 2>&1 + kill -9 "$1" >/dev/null 2>&1 }