summaryrefslogtreecommitdiffstats
path: root/scripts/lib/wic
diff options
context:
space:
mode:
authorLeonardo Augusto <leobsd@gmail.com>2019-02-08 08:35:12 -0800
committerRichard Purdie <richard.purdie@linuxfoundation.org>2019-02-14 11:20:55 +0000
commit89d4a8df074598cfb3a76e41db7c45d845afd961 (patch)
treeb4c9587b54b8a076ee5209f1e8ed3917a8851528 /scripts/lib/wic
parent5501f12ef605a874479a050c4a391f9610a1cebf (diff)
downloadpoky-89d4a8df074598cfb3a76e41db7c45d845afd961.tar.gz
scripts/lib/wic/engine: Fix cp's target path for ext* filesystems
Python subprocess' shell=True defaults to /bin/sh[1][2], which often refers to a POSIX-compliant shell. As the -e flag is not defined in the POSIX standard[3], some shells may interpret "-e" as the first argument to the echo command, causing the rest of the command line to fail. In this particular case, "echo -e 'cd {}'" is interpreted as "-e cd {}", which causes the first line of the command to fail, and causing cp to always place the source file in the filesystem's root. Replacing "echo -e" for a printf command makes this more portable. This issue only affects "wic cp" for ext* filesystems. [1] https://docs.python.org/2/library/subprocess.html [2] https://docs.python.org/3/library/subprocess.html [3] http://pubs.opengroup.org/onlinepubs/9699919799 (From OE-Core rev: 9a5b703d4e60c3ccdf80b5289b8e6fc91133fcde) Signed-off-by: Leonardo Augusto <leobsd@gmail.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts/lib/wic')
-rw-r--r--scripts/lib/wic/engine.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/lib/wic/engine.py b/scripts/lib/wic/engine.py
index 4662c665c0..97f5e848cc 100644
--- a/scripts/lib/wic/engine.py
+++ b/scripts/lib/wic/engine.py
@@ -331,7 +331,7 @@ class Disk:
331 def copy(self, src, pnum, path): 331 def copy(self, src, pnum, path):
332 """Copy partition image into wic image.""" 332 """Copy partition image into wic image."""
333 if self.partitions[pnum].fstype.startswith('ext'): 333 if self.partitions[pnum].fstype.startswith('ext'):
334 cmd = "echo -e 'cd {}\nwrite {} {}' | {} -w {}".\ 334 cmd = "printf 'cd {}\nwrite {} {}' | {} -w {}".\
335 format(path, src, os.path.basename(src), 335 format(path, src, os.path.basename(src),
336 self.debugfs, self._get_part_image(pnum)) 336 self.debugfs, self._get_part_image(pnum))
337 else: # fat 337 else: # fat