Re: [PATCH] hw/nvme: Clean up local variable shadowing in nvme_ns_init()

[Jesper Wendel Devantier]

Klaus Jensen <its@irrelevant.dk> writes:

> From: Klaus Jensen <k.jensen@samsung.com>
>
> Fix local variable shadowing in nvme_ns_init().
>
> Reported-by: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Klaus Jensen <k.jensen@samsung.com>
> ---
>  hw/nvme/ns.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/nvme/ns.c b/hw/nvme/ns.c
> index 44aba8f4d9cf..0eabcf5cf500 100644
> --- a/hw/nvme/ns.c
> +++ b/hw/nvme/ns.c
> @@ -107,7 +107,7 @@ static int nvme_ns_init(NvmeNamespace *ns, Error **errp)
>
>      ns->pif = ns->params.pif;
>
> -    static const NvmeLBAF lbaf[16] = {
> +    static const NvmeLBAF defaults[16] = {
>          [0] = { .ds =  9           },
>          [1] = { .ds =  9, .ms =  8 },
>          [2] = { .ds =  9, .ms = 16 },
> @@ -120,7 +120,7 @@ static int nvme_ns_init(NvmeNamespace *ns, Error **errp)
>
>      ns->nlbaf = 8;
>
> -    memcpy(&id_ns->lbaf, &lbaf, sizeof(lbaf));
> +    memcpy(&id_ns->lbaf, &defaults, sizeof(defaults));
>
>      for (i = 0; i < ns->nlbaf; i++) {
>          NvmeLBAF *lbaf = &id_ns->lbaf[i];
>
> ---
> base-commit: b55e4b9c0525560577384adfc6d30eb0daa8d7be
> change-id: 20230925-fix-local-shadowing-9606793e8ae9
>
> Best regards,
>

Reviewed-by: Jesper Wendel Devantier <j.devantier@samsung.com>