From 52868d5ad1ca2d55a085771290b9fecf7beb6e8f Mon Sep 17 00:00:00 2001 From: w1nst0n Date: Sun, 17 Jan 2021 18:38:31 +0100 Subject: [PATCH] Some minor corrections (shellcheck) --- CommonAddon | 4 +-- CommonInstaller | 34 +++++++++---------- build-package | 2 +- .../META-INF/com/google/android/update-binary | 14 ++++---- 4 files changed, 27 insertions(+), 27 deletions(-) diff --git a/CommonAddon b/CommonAddon index f18403b..68b0c79 100644 --- a/CommonAddon +++ b/CommonAddon @@ -53,8 +53,8 @@ EOF } ui_print () { - echo -n -e "ui_print ${1}\n" >> /proc/self/fd/${OUTFD} - echo -n -e "ui_print\n" >> /proc/self/fd/${OUTFD} + echo -e "ui_print ${1}" >> /proc/self/fd/${OUTFD} + echo -e "ui_print" >> /proc/self/fd/${OUTFD} } detect_outfd () { diff --git a/CommonInstaller b/CommonInstaller index 75b5271..6680d01 100644 --- a/CommonInstaller +++ b/CommonInstaller @@ -77,16 +77,16 @@ print_info () { echo "${@}" decho "${@}" else - echo -n -e "ui_print ${@}\n" >> /proc/self/fd/${OUTFD} - echo -n -e "ui_print\n" >> /proc/self/fd/${OUTFD} + echo -e "ui_print ${@}" >> /proc/self/fd/${OUTFD} + echo -e "ui_print" >> /proc/self/fd/${OUTFD} fi } -show_progress() { +show_progress () { ${BOOTMODE} || echo "progress ${1} ${2}" >> /proc/self/fd/${OUTFD} } -set_progress() { +set_progress () { ${BOOTMODE} || echo "set_progress ${1}" >> /proc/self/fd/${OUTFD} } @@ -161,11 +161,11 @@ detect_bootmode () { # Taken from Magisk, grep build_props / /proc/cmdline for value ########################################################################################## -grep_prop() { +grep_prop () { sed -n "s/^${1}=//p" /system/build.prop ${2} | head -n 1 } -grep_cmdline() { +grep_cmdline () { local REGEX="s/^${1}=//p" sed -E 's/ +/\n/g' /proc/cmdline | \ sed -n "${REGEX}" 2>/dev/null @@ -185,7 +185,7 @@ set_perm () { fi } -set_perm_recursive() { +set_perm_recursive () { find ${1} -type d 2>/dev/null | while read dir; do set_perm ${dir} ${2} ${3} ${4} ${6} done @@ -234,13 +234,13 @@ toupper() { } find_block () { - local block tmp partname devname device + local block partname devname device block=${1}${SLOT} device=$(find /dev/block \( -type b -o -type c -o -type l \) -name ${block} | head -n 1) - if [ ! -z ${device} ]; then - readlink -f ${device} + if [ -n "$device" ]; then + readlink -f "$device" return 0 fi @@ -254,8 +254,8 @@ find_block () { done device=$(find /dev \( -type b -o -type c -o -type l \) -maxdepth 1 -iname ${block} | head -n 1) - if [ ! -z ${device} ]; then - readlink -f ${device} + if [ -n "$device" ]; then + readlink -f "$device" return 0 fi @@ -1505,8 +1505,8 @@ install_addond () { installinfo_add () { if [ "${MODE}" = "SYSTEM" ]; then - for file in ${@}; do - echo ${file} >> ${NANODROID_LIST} + for file in "${@}"; do + echo "${file}" >> ${NANODROID_LIST} done fi } @@ -1530,7 +1530,7 @@ nanodroid_install_file () { [ -z "${source}" ] && error "file: ${file} not found" # strip off .${BIN_ARCH} from final filename (if any) - xfile=$(basename ${file} .${BIN_ARCH}) + xfile=$(basename "${file}" .${BIN_ARCH}) decho " installing file: ${xfile}" @@ -1627,7 +1627,7 @@ nanodroid_install_apk () { nanodroid_install_lib "${dest}/${dest_apk}" "${2}" - [ "${dir}" == "system/priv-app" ] && create_privapp_permissions_whitelist ${app} + [ "${dir}" = "system/priv-app" ] && create_privapp_permissions_whitelist "$app" set_perm_data -r "${dest}" } @@ -1638,7 +1638,7 @@ nanodroid_install_apk () { nanodroid_install_lib () { apk="${1}" - dest="$(dirname ${1})" + dest="$(dirname "$1")" multilib="${2}" unset libd diff --git a/build-package b/build-package index f71c8f5..2d0187f 100755 --- a/build-package +++ b/build-package @@ -47,7 +47,7 @@ done [[ -z ${1} ]] && show_help -for opt in ${@}; do +for opt in "$@"; do case ${opt} in full) check_nanodroid || error "run 'build-package pull' first!" diff --git a/uninstaller/META-INF/com/google/android/update-binary b/uninstaller/META-INF/com/google/android/update-binary index 20c3274..8cb9649 100644 --- a/uninstaller/META-INF/com/google/android/update-binary +++ b/uninstaller/META-INF/com/google/android/update-binary @@ -12,8 +12,8 @@ detect_bootmode () { } ui_print() { - echo -n -e "ui_print $1\n" >> /proc/self/fd/$OUTFD - echo -n -e "ui_print\n" >> /proc/self/fd/$OUTFD + echo -e "ui_print $1" >> /proc/self/fd/$OUTFD + echo -e "ui_print" >> /proc/self/fd/$OUTFD } grep_prop() { @@ -102,13 +102,13 @@ toupper() { } find_block () { - local block tmp partname devname device + local block partname devname device block=${1}${SLOT} device=$(find /dev/block \( -type b -o -type c -o -type l \) -name ${block} | head -n 1) - if [ ! -z ${device} ]; then - readlink -f ${device} + if [ -n "$device" ]; then + readlink -f "$device" return 0 fi @@ -122,8 +122,8 @@ find_block () { done device=$(find /dev \( -type b -o -type c -o -type l \) -maxdepth 1 -iname ${block} | head -n 1) - if [ ! -z ${device} ]; then - readlink -f ${device} + if [ -n "$device" ]; then + readlink -f "$device" return 0 fi