Commit de5dca1b authored by Dmitry Fleytman's avatar Dmitry Fleytman Committed by Peter Maydell
Browse files

e1000e: Fix build with gcc 4.6.3 and ust tracing 



This patch fixes used-uninitialized false
positive while compiling with ust tracing
backend plus gcc 4.6.3:

hw/net/e1000e.c: In function ‘e1000e_io_write’:
hw/net/e1000e.c:170:39: error: ‘idx’ may be used uninitialized in this function [-Werror=uninitialized]
hw/net/e1000e.c: In function ‘e1000e_io_read’:
hw/net/e1000e.c:145:35: error: ‘idx’ may be used uninitialized in this function [-Werror=uninitialized]
cc1: all warnings being treated as errors
make: *** [hw/net/e1000e.o] Error 1

Signed-off-by: default avatarDmitry Fleytman <dmitry@daynix.com>
Acked-by: default avatarJason Wang <jasowang@redhat.com>
Message-id: 1465023763-10773-1-git-send-email-dmitry@daynix.com
Signed-off-by: default avatarPeter Maydell <peter.maydell@linaro.org>
parent 6b3532b2

hw/net/e1000e.c

+2 −2
Original line number Diff line number Diff line
@@ -133,7 +133,7 @@ static uint64_t 
e1000e_io_read(void *opaque, hwaddr addr, unsigned size)

{

    E1000EState *s = opaque;

    uint32_t idx;

    uint32_t idx = 0;

    uint64_t val;



    switch (addr) {
@@ -158,7 +158,7 @@ e1000e_io_write(void *opaque, hwaddr addr, 
                uint64_t val, unsigned size)

{

    E1000EState *s = opaque;

    uint32_t idx;

    uint32_t idx = 0;



    switch (addr) {

    case E1000_IOADDR: