Re: [PATCH v4 34/35] bloblist: Fix use of uninitialized variable

From: Simon Glass
Date: Thu Sep 19 2024 - 10:11:06 EST


On Wed, 18 Sept 2024 at 17:28, Patrick Rudolph
<patrick.rudolph@xxxxxxxxxxxxx> wrote:
>
> Initialize addr to zero which allows to build on the CI
> which is more strict.
>
> Signed-off-by: Patrick Rudolph <patrick.rudolph@xxxxxxxxxxxxx>
> ---
> common/bloblist.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>

Reviewed-by: Simon Glass <sjg@xxxxxxxxxxxx>

New patch, I believe


> diff --git a/common/bloblist.c b/common/bloblist.c
> index 2008ab4d25..cf1a3b8b62 100644
> --- a/common/bloblist.c
> +++ b/common/bloblist.c
> @@ -499,7 +499,7 @@ int bloblist_init(void)
> {
> bool fixed = IS_ENABLED(CONFIG_BLOBLIST_FIXED);
> int ret = -ENOENT;
> - ulong addr, size;
> + ulong addr = 0, size;
> /*
> * If U-Boot is not in the first phase, an existing bloblist must be
> * at a fixed address.
> --
> 2.46.0
>