On 3/21/20 10:41 AM, Philippe Mathieu-Daudé wrote: > Fix warning reported by Clang static code analyzer: > > CC hw/ide/sii3112.o > hw/ide/sii3112.c:204:9: warning: Value stored to 'val' is never read > val = 0; > ^ ~ > > Fixes: a9dd6604 > Reported-by: Clang Static Analyzer > Reviewed-by: BALATON Zoltan <bala...@eik.bme.hu> > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> > --- > v2: Fix the correct function (Aleksandar review) > --- > hw/ide/sii3112.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/hw/ide/sii3112.c b/hw/ide/sii3112.c > index 06605d7af2..b2ff6dd6d9 100644 > --- a/hw/ide/sii3112.c > +++ b/hw/ide/sii3112.c > @@ -42,7 +42,7 @@ static uint64_t sii3112_reg_read(void *opaque, hwaddr addr, > unsigned int size) > { > SiI3112PCIState *d = opaque; > - uint64_t val = 0; > + uint64_t val; > > switch (addr) { > case 0x00: > @@ -126,6 +126,7 @@ static uint64_t sii3112_reg_read(void *opaque, hwaddr > addr, > break; > default: > val = 0; > + break; > } > trace_sii3112_read(size, addr, val); > return val; > @@ -201,7 +202,7 @@ static void sii3112_reg_write(void *opaque, hwaddr addr, > d->regs[1].sien = (val >> 16) & 0x3eed; > break; > default: > - val = 0; > + break; > } > } > > Acked-by: John Snow <js...@redhat.com>