From: Harald Hoyer <harald@redhat.com>
Date: Fri, 26 Mar 2021 09:29:11 +0000 (+0100)
Subject: fix(watchdog-modules): shellcheck for modules.d/04watchdog-modules
X-Git-Tag: 054~263
X-Git-Url: http://git.ipfire.org/gitweb/gitweb.cgi?a=commitdiff_plain;h=95729721ccb29a3b98fa38d7fe0817f71f868f5d;p=thirdparty%2Fdracut.git

fix(watchdog-modules): shellcheck for modules.d/04watchdog-modules
---

diff --git a/modules.d/04watchdog-modules/.shchkdir b/modules.d/04watchdog-modules/.shchkdir
new file mode 100644
index 000000000..e69de29bb
diff --git a/modules.d/04watchdog-modules/module-setup.sh b/modules.d/04watchdog-modules/module-setup.sh
index 705592fac..2a94e3464 100755
--- a/modules.d/04watchdog-modules/module-setup.sh
+++ b/modules.d/04watchdog-modules/module-setup.sh
@@ -17,13 +17,13 @@ install() {
 
 installkernel() {
     local -A _drivers
-    local _alldrivers _wdtdrv _wdtppath _dir
+    local _wdtdrv
 
     for _wd in /sys/class/watchdog/*; do
-        ! [ -e $_wd ] && continue
-        _wdtdrv=$(get_dev_module $_wd)
+        ! [ -e "$_wd" ] && continue
+        _wdtdrv=$(get_dev_module "$_wd")
         if [[ $_wdtdrv ]]; then
-            instmods $_wdtdrv
+            instmods "$_wdtdrv"
             for i in $_wdtdrv; do
                 _drivers[$i]=1
             done
@@ -31,8 +31,11 @@ installkernel() {
     done
 
     # ensure that watchdog module is loaded as early as possible
-    _alldrivers="${!_drivers[*]}"
-    [[ $_alldrivers ]] && echo "rd.driver.pre=${_alldrivers// /,}" > ${initdir}/etc/cmdline.d/00-watchdog.conf
-
+    if [[ ${!_drivers[*]} ]]; then
+        echo "rd.driver.pre=\"$(
+            IFS=,
+            echo "${!_drivers[*]}"
+        )\"" > "${initdir}"/etc/cmdline.d/00-watchdog.conf
+    fi
     return 0
 }