[PATCH v3 10/13] env: Make return value of env_get_f() behave like sprintf() on success

Simon Glass sjg at chromium.org
Thu Oct 21 20:49:07 CEST 2021


From: Marek Behún <marek.behun at nic.cz>

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 <marek.behun at nic.cz>
Reviewed-by: Simon Glass <sjg at chromium.org>
---
 cmd/nvedit.c  | 8 +++++---
 include/env.h | 6 ++----
 2 files changed, 7 insertions(+), 7 deletions(-)

Applied to u-boot-dm, thanks!


More information about the U-Boot mailing list