Re: [Qemu-devel] [PATCH] sdhci: fix incorrect use of Error *

[Peter Maydell]

On 20 March 2018 at 15:13, Paolo Bonzini <address@hidden> wrote:
> Detected by Coverity (CID 1386072, 1386073, 1386076, 1386077).  local_err
> was unused, and this made the static analyzer unhappy.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
>  hw/sd/sdhci.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/sd/sdhci.c b/hw/sd/sdhci.c
> index 1b828b104d..63c44a4ee8 100644
> --- a/hw/sd/sdhci.c
> +++ b/hw/sd/sdhci.c
> @@ -1474,7 +1474,7 @@ static void sdhci_pci_realize(PCIDevice *dev, Error 
> **errp)
>      Error *local_err = NULL;
>
>      sdhci_initfn(s);
> -    sdhci_common_realize(s, errp);
> +    sdhci_common_realize(s, &local_err);
>      if (local_err) {
>          error_propagate(errp, local_err);
>          return;
> @@ -1556,7 +1556,7 @@ static void sdhci_sysbus_realize(DeviceState *dev, 
> Error ** errp)
>      SysBusDevice *sbd = SYS_BUS_DEVICE(dev);
>      Error *local_err = NULL;
>
> -    sdhci_common_realize(s, errp);
> +    sdhci_common_realize(s, &local_err);
>      if (local_err) {
>          error_propagate(errp, local_err);
>          return;
> --
> 2.16.2

Applied to target-arm.next, thanks.

-- PMM