commit 74de807f794ac5201b2b3c38ddadeef84a676a97
author Paolo Bonzini <pbonzini@redhat.com> Mon Oct 19 16:08:38 2015 +0200
committer Michael Tokarev <mjt@tls.msk.ru> Fri Nov 06 15:42:38 2015 +0300
tree 94da0984128a734804b2695102c36e227e6ec161
parent 258133bda9a6f22ba436ef9b63b7c086cc80190b

target-alpha: fix uninitialized variable

I am not sure why the compiler does not catch it.  There is no
semantic change since gen_excp returns EXIT_NORETURN, but the
old code is wrong.

Reported by Coverity.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>

target-alpha/translate.c

diff --git a/target-alpha/translate.c b/target-alpha/translate.c
index 87950c6..9909c70 100644
--- a/target-alpha/translate.c
+++ b/target-alpha/translate.c
@@ -2916,7 +2916,7 @@
         num_insns++;
 
         if (unlikely(cpu_breakpoint_test(cs, ctx.pc, BP_ANY))) {
-            gen_excp(&ctx, EXCP_DEBUG, 0);
+            ret = gen_excp(&ctx, EXCP_DEBUG, 0);
             /* The address covered by the breakpoint must be included in
                [tb->pc, tb->pc + tb->size) in order to for it to be
                properly cleared -- thus we increment the PC here so that