[U-Boot] [PATCH] scripts: setlocalversion: safely extract variables from auto.conf using awk
Tom Rini
trini at konsulko.com
Thu Aug 10 01:42:16 UTC 2017
On Tue, Aug 08, 2017 at 03:37:03PM +0200, Philipp Tomsich wrote:
> Moving SPL_LDSCRIPT to Kconfig triggered an unfortunate attempt of
> command substitution, as the sourced auto.conf may include $(ARCH)
> which tries to execute a command 'ARCH'.
> This showed up as a warning similar to the following:
> include/config/auto.conf: line 209: ARCH: command not found
>
> This change does no longer attempt to source auto.conf, but rather
> passes it through awk to retrieve the values for CONFIG_LOCALVERSION
> and CONFIG_LOCALVERSION_AUTO. This will also mitigate the risk of
> unintended command substitution.
>
> Signed-off-by: Philipp Tomsich <philipp.tomsich at theobroma-systems.com>
> Reported-by: Andy Yan <andy.yan at rock-chips.com>
>
Reviewed-by: Tom Rini <trini at konsulko.com>
I do wonder if we can get this into the kernel as well.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170809/aadf262c/attachment.sig>
More information about the U-Boot
mailing list