* [PATCH] env: Make _init() expect _INVALID when _IS_NOWHERE
@ 2021-08-12 15:28 Pierre-Clément Tosi
2021-08-12 15:38 ` Marek Vasut
2021-09-24 2:40 ` Tom Rini
0 siblings, 2 replies; 5+ messages in thread
From: Pierre-Clément Tosi @ 2021-08-12 15:28 UTC (permalink / raw)
To: u-boot
Cc: Marek Vasut, Wolfgang Denk, Joe Hershberger, Pierre-Clément Tosi
Avoid applying the "fix" introduced by commit 5557eec01cbf ("env: Fix
invalid env handling in env_init()") to the environment "nowhere".
This is necessary as that commit, by setting the return value of
env_init() to -ENOENT if gd->env_valid is ENV_INVALID, forces that
function to reset gd->env_valid to ENV_VALID. By doing so, it breaks the
assumption (required by ENV_IS_NOWHERE) that gd->env_valid must be
ENV_INVALID.
This, in turn, results in env_relocate() calling env_load() (it should
not), which itself, calls U_BOOT_ENV_LOCATION(nowhere).load() i.e.
env_nowhere_load(). That function, being implemented under the
assumption mentioned above, calls env_set_default(), which in turn,
seeing that gd->env_valid is ENV_VALID (it should not), tries to
dereference whatever lies in gd->env_addr (most likely garbage), leading
to a faulty memory access.
Note that other env_locations might be concerned by this bug but that
this commit only intends to fix it for when ENV_IS_NOWHERE.
Fixes: 5557eec01cbf ("env: Fix invalid env handling in env_init()")
Signed-off-by: Pierre-Clément Tosi <ptosi@google.com>
---
env/env.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/env/env.c b/env/env.c
index e534008006..0a0f234747 100644
--- a/env/env.c
+++ b/env/env.c
@@ -336,7 +336,7 @@ int env_init(void)
debug("%s: Environment %s init done (ret=%d)\n", __func__,
drv->name, ret);
- if (gd->env_valid == ENV_INVALID)
+ if (gd->env_valid == ENV_INVALID && drv->location != ENVL_NOWHERE)
ret = -ENOENT;
}
--
2.32.0.605.g8dce9f2422-goog
--
Pierre
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] env: Make _init() expect _INVALID when _IS_NOWHERE
2021-08-12 15:28 [PATCH] env: Make _init() expect _INVALID when _IS_NOWHERE Pierre-Clément Tosi
@ 2021-08-12 15:38 ` Marek Vasut
2021-09-24 2:40 ` Tom Rini
1 sibling, 0 replies; 5+ messages in thread
From: Marek Vasut @ 2021-08-12 15:38 UTC (permalink / raw)
To: Pierre-Clément Tosi, u-boot
Cc: Wolfgang Denk, Joe Hershberger, Kunihiko Hayashi
On 8/12/21 5:28 PM, Pierre-Clément Tosi wrote:
> Avoid applying the "fix" introduced by commit 5557eec01cbf ("env: Fix
> invalid env handling in env_init()") to the environment "nowhere".
>
> This is necessary as that commit, by setting the return value of
> env_init() to -ENOENT if gd->env_valid is ENV_INVALID, forces that
> function to reset gd->env_valid to ENV_VALID. By doing so, it breaks the
> assumption (required by ENV_IS_NOWHERE) that gd->env_valid must be
> ENV_INVALID.
>
> This, in turn, results in env_relocate() calling env_load() (it should
> not), which itself, calls U_BOOT_ENV_LOCATION(nowhere).load() i.e.
> env_nowhere_load(). That function, being implemented under the
> assumption mentioned above, calls env_set_default(), which in turn,
> seeing that gd->env_valid is ENV_VALID (it should not), tries to
> dereference whatever lies in gd->env_addr (most likely garbage), leading
> to a faulty memory access.
>
> Note that other env_locations might be concerned by this bug but that
> this commit only intends to fix it for when ENV_IS_NOWHERE.
>
> Fixes: 5557eec01cbf ("env: Fix invalid env handling in env_init()")
> Signed-off-by: Pierre-Clément Tosi <ptosi@google.com>
Such patch was already posted, see the patch and the discussion, in the
end there was a different fix for the relocation code:
https://patchwork.ozlabs.org/project/uboot/patch/1620828554-24013-1-git-send-email-hayashi.kunihiko@socionext.com/
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] env: Make _init() expect _INVALID when _IS_NOWHERE
2021-08-12 15:28 [PATCH] env: Make _init() expect _INVALID when _IS_NOWHERE Pierre-Clément Tosi
2021-08-12 15:38 ` Marek Vasut
@ 2021-09-24 2:40 ` Tom Rini
2021-09-24 3:04 ` Marek Vasut
1 sibling, 1 reply; 5+ messages in thread
From: Tom Rini @ 2021-09-24 2:40 UTC (permalink / raw)
To: Pierre-Clément Tosi
Cc: u-boot, Marek Vasut, Wolfgang Denk, Joe Hershberger
[-- Attachment #1: Type: text/plain, Size: 1297 bytes --]
On Thu, Aug 12, 2021 at 03:28:31PM +0000, Pierre-Clément Tosi wrote:
> Avoid applying the "fix" introduced by commit 5557eec01cbf ("env: Fix
> invalid env handling in env_init()") to the environment "nowhere".
>
> This is necessary as that commit, by setting the return value of
> env_init() to -ENOENT if gd->env_valid is ENV_INVALID, forces that
> function to reset gd->env_valid to ENV_VALID. By doing so, it breaks the
> assumption (required by ENV_IS_NOWHERE) that gd->env_valid must be
> ENV_INVALID.
>
> This, in turn, results in env_relocate() calling env_load() (it should
> not), which itself, calls U_BOOT_ENV_LOCATION(nowhere).load() i.e.
> env_nowhere_load(). That function, being implemented under the
> assumption mentioned above, calls env_set_default(), which in turn,
> seeing that gd->env_valid is ENV_VALID (it should not), tries to
> dereference whatever lies in gd->env_addr (most likely garbage), leading
> to a faulty memory access.
>
> Note that other env_locations might be concerned by this bug but that
> this commit only intends to fix it for when ENV_IS_NOWHERE.
>
> Fixes: 5557eec01cbf ("env: Fix invalid env handling in env_init()")
> Signed-off-by: Pierre-Clément Tosi <ptosi@google.com>
Applied to u-boot/next, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] env: Make _init() expect _INVALID when _IS_NOWHERE
2021-09-24 2:40 ` Tom Rini
@ 2021-09-24 3:04 ` Marek Vasut
2021-09-24 11:41 ` Tom Rini
0 siblings, 1 reply; 5+ messages in thread
From: Marek Vasut @ 2021-09-24 3:04 UTC (permalink / raw)
To: Tom Rini, Pierre-Clément Tosi; +Cc: u-boot, Wolfgang Denk, Joe Hershberger
On 9/24/21 4:40 AM, Tom Rini wrote:
> On Thu, Aug 12, 2021 at 03:28:31PM +0000, Pierre-Clément Tosi wrote:
>
>> Avoid applying the "fix" introduced by commit 5557eec01cbf ("env: Fix
>> invalid env handling in env_init()") to the environment "nowhere".
>>
>> This is necessary as that commit, by setting the return value of
>> env_init() to -ENOENT if gd->env_valid is ENV_INVALID, forces that
>> function to reset gd->env_valid to ENV_VALID. By doing so, it breaks the
>> assumption (required by ENV_IS_NOWHERE) that gd->env_valid must be
>> ENV_INVALID.
>>
>> This, in turn, results in env_relocate() calling env_load() (it should
>> not), which itself, calls U_BOOT_ENV_LOCATION(nowhere).load() i.e.
>> env_nowhere_load(). That function, being implemented under the
>> assumption mentioned above, calls env_set_default(), which in turn,
>> seeing that gd->env_valid is ENV_VALID (it should not), tries to
>> dereference whatever lies in gd->env_addr (most likely garbage), leading
>> to a faulty memory access.
>>
>> Note that other env_locations might be concerned by this bug but that
>> this commit only intends to fix it for when ENV_IS_NOWHERE.
>>
>> Fixes: 5557eec01cbf ("env: Fix invalid env handling in env_init()")
>> Signed-off-by: Pierre-Clément Tosi <ptosi@google.com>
>
> Applied to u-boot/next, thanks!
Please drop this, quote from my previous reply to this patch:
"
Such patch was already posted, see the patch and the discussion, in the
end there was a different fix for the relocation code:
https://patchwork.ozlabs.org/project/uboot/patch/1620828554-24013-1-git-send-email-hayashi.kunihiko@socionext.com/
"
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] env: Make _init() expect _INVALID when _IS_NOWHERE
2021-09-24 3:04 ` Marek Vasut
@ 2021-09-24 11:41 ` Tom Rini
0 siblings, 0 replies; 5+ messages in thread
From: Tom Rini @ 2021-09-24 11:41 UTC (permalink / raw)
To: Marek Vasut
Cc: Pierre-Clément Tosi, u-boot, Wolfgang Denk, Joe Hershberger
[-- Attachment #1: Type: text/plain, Size: 1972 bytes --]
On Fri, Sep 24, 2021 at 05:04:27AM +0200, Marek Vasut wrote:
> On 9/24/21 4:40 AM, Tom Rini wrote:
> > On Thu, Aug 12, 2021 at 03:28:31PM +0000, Pierre-Clément Tosi wrote:
> >
> > > Avoid applying the "fix" introduced by commit 5557eec01cbf ("env: Fix
> > > invalid env handling in env_init()") to the environment "nowhere".
> > >
> > > This is necessary as that commit, by setting the return value of
> > > env_init() to -ENOENT if gd->env_valid is ENV_INVALID, forces that
> > > function to reset gd->env_valid to ENV_VALID. By doing so, it breaks the
> > > assumption (required by ENV_IS_NOWHERE) that gd->env_valid must be
> > > ENV_INVALID.
> > >
> > > This, in turn, results in env_relocate() calling env_load() (it should
> > > not), which itself, calls U_BOOT_ENV_LOCATION(nowhere).load() i.e.
> > > env_nowhere_load(). That function, being implemented under the
> > > assumption mentioned above, calls env_set_default(), which in turn,
> > > seeing that gd->env_valid is ENV_VALID (it should not), tries to
> > > dereference whatever lies in gd->env_addr (most likely garbage), leading
> > > to a faulty memory access.
> > >
> > > Note that other env_locations might be concerned by this bug but that
> > > this commit only intends to fix it for when ENV_IS_NOWHERE.
> > >
> > > Fixes: 5557eec01cbf ("env: Fix invalid env handling in env_init()")
> > > Signed-off-by: Pierre-Clément Tosi <ptosi@google.com>
> >
> > Applied to u-boot/next, thanks!
>
> Please drop this, quote from my previous reply to this patch:
>
> "
> Such patch was already posted, see the patch and the discussion, in the end
> there was a different fix for the relocation code:
>
> https://patchwork.ozlabs.org/project/uboot/patch/1620828554-24013-1-git-send-email-hayashi.kunihiko@socionext.com/
> "
I'll drop this. FWIW, my reading of the previous patch, and then this
being posted afterwards is why I had applied this patch.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-09-24 11:41 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-12 15:28 [PATCH] env: Make _init() expect _INVALID when _IS_NOWHERE Pierre-Clément Tosi
2021-08-12 15:38 ` Marek Vasut
2021-09-24 2:40 ` Tom Rini
2021-09-24 3:04 ` Marek Vasut
2021-09-24 11:41 ` Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.