On Wed, 13 Oct 2021 at 09:46, Marek Behún <[email protected]> wrote:
>
> From: Marek Behún <[email protected]>
>
> Currently the env_get_f() function's return value behaves weirdly: it
> returns the number of bytes written into `buf`, but whether this is
> excluding the terminating NULL-byte or including it depends on whether
> there was enough space in `buf`.
>
> Change the function to always return the actual length of the value of
> the environment variable (excluding the terminating NULL-byte) on
> success. This makes it behave like sprintf().
>
> All users of this function in U-Boot are compatible with this change.
>
> Signed-off-by: Marek Behún <[email protected]>
> ---
>  cmd/nvedit.c  | 8 +++++---
>  include/env.h | 6 ++----
>  2 files changed, 7 insertions(+), 7 deletions(-)

Reviewed-by: Simon Glass <[email protected]>

Reply via email to