[PATCH v3 1/1] input: avoid NULL dereference

Tom Rini trini at konsulko.com
Wed Oct 11 20:38:15 CEST 2023


On Tue, Oct 03, 2023 at 03:09:01AM +0200, Heinrich Schuchardt wrote:

> Before using the result of env_get("stdin") we must check if it is NULL.
> 
> Avoid #if. This resolves the -Wunused-but-set-variable issue and we don't
> need a dummy assignment in the else branch. Anyway this warning is
> disabled in the Makefile.
> 
> For sake of readability use an early return after the configuration check.
> 
> Checking CONFIG_SPL_BUILD is incorrect as env_get() is only defined if
> CONFIG_$(SPL_TPL)ENV_SUPPORT=y.
> 
> Fixes: 985ca3945fa3 ("spl: input: Allow input in SPL and TPL")
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> Reviewed-by: Tom Rini <trini at konsulko.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20231011/cef3d0c4/attachment-0001.sig>


More information about the U-Boot mailing list