From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34263) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b95br-00068V-OA for qemu-devel@nongnu.org; Sat, 04 Jun 2016 03:03:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1b95bn-0000Ew-IG for qemu-devel@nongnu.org; Sat, 04 Jun 2016 03:03:02 -0400 Received: from mail-wm0-x241.google.com ([2a00:1450:400c:c09::241]:33255) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b95bn-0000Ei-AK for qemu-devel@nongnu.org; Sat, 04 Jun 2016 03:02:59 -0400 Received: by mail-wm0-x241.google.com with SMTP id a136so3851130wme.0 for ; Sat, 04 Jun 2016 00:02:58 -0700 (PDT) From: Dmitry Fleytman Date: Sat, 4 Jun 2016 10:02:43 +0300 Message-Id: <1465023763-10773-1-git-send-email-dmitry@daynix.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [Qemu-devel] [PATCH] e1000e: Fix build with gcc 4.6.3 and ust tracing List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Jason Wang , Yan Vugenfirer , Leonid Bloch , Shmulik Ladkani , Peter Maydell 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: Dmitry Fleytman --- hw/net/e1000e.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/net/e1000e.c b/hw/net/e1000e.c index 61bcbb6..692283f 100644 --- a/hw/net/e1000e.c +++ b/hw/net/e1000e.c @@ -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: -- 2.5.5