env: Fix invalid env handling in env_init()
authorMarek Vasut <marex@denx.de>
Wed, 20 Jan 2021 14:45:16 +0000 (15:45 +0100)
committerTom Rini <trini@konsulko.com>
Fri, 16 Apr 2021 17:37:07 +0000 (13:37 -0400)
commit5557eec01cbfb0e415775434f29542dffb1a4423
treeed49bbeacb8ee2401f35cbce69e56828337cb028
parent2339f01af6478f93aada713fa33b5e04c3497724
env: Fix invalid env handling in env_init()

This fixes the case where there are multiple environment drivers, one of
them is the default environment one, and it is followed by an environment
driver which does not implement .init() callback. The default environment
driver sets gd->env_valid to ENV_INVALID and returns 0 from its .init()
callback implementation, which is valid behavior for default environment.

Since the subsequent environment driver does not implement .init(), it
also does not modify the $ret variable in the loop. Therefore, the loop
is exited with gd->env_valid=ENV_INVALID and ret=0, which means that the
code further down in env_init() will not reset the environment to the
default one, which is incorrect.

This patch sets the $ret variable back to -ENOENT in case the env_valid
is set to ENV_INVALID by an environment driver, so that the environment
would be correctly reset back to default one, unless a subsequent driver
loads a valid environment.

Signed-off-by: Marek Vasut <marex@denx.de>
Tested-By: Tim Harvey <tharvey@gateworks.com>
env/env.c