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

fix(kernel-modules): shellcheck for modules.d/90kernel-modules
---

diff --git a/modules.d/90kernel-modules/.shchkdir b/modules.d/90kernel-modules/.shchkdir
new file mode 100644
index 000000000..e69de29bb
diff --git a/modules.d/90kernel-modules/insmodpost.sh b/modules.d/90kernel-modules/insmodpost.sh
index 4362f0a75..a7ab05bdb 100755
--- a/modules.d/90kernel-modules/insmodpost.sh
+++ b/modules.d/90kernel-modules/insmodpost.sh
@@ -6,7 +6,7 @@ for modlist in $(getargs rd.driver.post -d rdinsmodpost=); do
     (
         IFS=,
         for m in $modlist; do
-            modprobe $m
+            modprobe "$m"
         done
     )
 done
diff --git a/modules.d/90kernel-modules/module-setup.sh b/modules.d/90kernel-modules/module-setup.sh
index ed52798c1..bbafb289a 100755
--- a/modules.d/90kernel-modules/module-setup.sh
+++ b/modules.d/90kernel-modules/module-setup.sh
@@ -6,22 +6,17 @@ installkernel() {
     local _hostonly_drvs
 
     find_kernel_modules_external() {
-        local _OLDIFS
-        local external_pattern="^/"
+        local a
 
         [[ -f "$srcmods/modules.dep" ]] || return 0
 
-        _OLDIFS=$IFS
-        IFS=:
-        while read a rest; do
-            [[ $a =~ $external_pattern ]] || continue
-            printf "%s\n" "$a"
+        while IFS=: read -r a _ || [[ $a ]]; do
+            [[ $a =~ ^/ ]] && printf "%s\n" "$a"
         done < "$srcmods/modules.dep"
-        IFS=$_OLDIFS
     }
 
     record_block_dev_drv() {
-        for _mod in $(get_dev_module /dev/block/$1); do
+        for _mod in $(get_dev_module /dev/block/"$1"); do
             [[ " $_hostonly_drvs " != *$_mod* ]] && _hostonly_drvs+=" $_mod"
         done
         [[ "$_hostonly_drvs" ]] && return 0
@@ -29,7 +24,7 @@ installkernel() {
     }
 
     install_block_modules_strict() {
-        hostonly='' instmods $_hostonly_drvs
+        hostonly='' instmods "$_hostonly_drvs"
     }
 
     install_block_modules() {
@@ -48,8 +43,7 @@ installkernel() {
             ehci-hcd ehci-pci ehci-platform \
             ohci-hcd ohci-pci \
             uhci-hcd \
-            xhci-hcd xhci-pci xhci-plat-hcd \
-            ${NULL}
+            xhci-hcd xhci-pci xhci-plat-hcd
 
         hostonly=$(optional_hostonly) instmods \
             "=drivers/hid" \
@@ -58,8 +52,7 @@ installkernel() {
             "=drivers/input/keyboard" \
             "=drivers/pci/host" \
             "=drivers/pci/controller" \
-            "=drivers/pinctrl" \
-            ${NULL}
+            "=drivers/pinctrl"
 
         instmods \
             yenta_socket \
@@ -96,8 +89,7 @@ installkernel() {
                 "=drivers/usb/misc" \
                 "=drivers/usb/musb" \
                 "=drivers/usb/phy" \
-                "=drivers/scsi/hisi_sas" \
-                ${NULL}
+                "=drivers/scsi/hisi_sas"
         fi
 
         find_kernel_modules_external | instmods
diff --git a/modules.d/90kernel-modules/parse-kernel.sh b/modules.d/90kernel-modules/parse-kernel.sh
index f1dd72f88..d14f912df 100755
--- a/modules.d/90kernel-modules/parse-kernel.sh
+++ b/modules.d/90kernel-modules/parse-kernel.sh
@@ -13,7 +13,7 @@ for i in $(getargs rd.driver.pre -d rdloaddriver=); do
     (
         IFS=,
         for p in $i; do
-            modprobe $p 2>&1 | vinfo
+            modprobe "$p" 2>&1 | vinfo
         done
     )
 done