[PATCH RESEND v2 14/16] copy-firmware: quote the output of dirname

From: Emil Velikov via B4 Relay <devnull+emil.l.velikov.gmail.com@kernel.org>
To: linux-firmware@kernel.org
Cc: Josh Boyer <jwboyer@kernel.org>, Adam Sampson <ats@offog.org>,
	 David Woodhouse <dwmw2@infradead.org>,
	 Emil Velikov <emil.l.velikov@gmail.com>
Subject: [PATCH RESEND v2 14/16] copy-firmware: quote the output of dirname
Date: Wed, 01 Mar 2023 18:56:28 +0000	[thread overview]
Message-ID: <20230301-fixes-and-compression-v2-14-e2b71974e842@gmail.com> (raw)
In-Reply-To: <20230301-fixes-and-compression-v2-0-e2b71974e842@gmail.com>

From: Emil Velikov <emil.l.velikov@gmail.com>

As highlighted by shellcheck - as-is one could get word splitting. That
isn't an issue, yet, since none of the in-tree directories have space in
their name.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
 copy-firmware.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/copy-firmware.sh b/copy-firmware.sh
index 722227f..5a5e1a8 100755
--- a/copy-firmware.sh
+++ b/copy-firmware.sh
@@ -27,13 +27,13 @@ done
 
 grep '^File:' WHENCE | sed -e 's/^File: *//g' | while read f; do
     $verbose "copying file $f"
-    install -d "$destdir"/$(dirname "$f")
+    install -d "$destdir/$(dirname "$f")"
     cp "$f" "$destdir/$f"
 done
 
 grep -E '^Link:' WHENCE | sed -e 's/^Link: *//g;s/-> //g' | while read f d; do
     $verbose "creating link $f -> $d"
-    install -d "$destdir"/$(dirname "$f")
+    install -d "$destdir/$(dirname "$f")"
     ln -s "$d" "$destdir/$f"
 done
 

-- 
2.39.2

