diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2015-10-19 16:08:38 +0200 |
|---|---|---|
| committer | Michael Tokarev <mjt@tls.msk.ru> | 2015-11-06 15:42:38 +0300 |
| commit | 74de807f794ac5201b2b3c38ddadeef84a676a97 (patch) | |
| tree | 94da0984128a734804b2695102c36e227e6ec161 | |
| parent | 258133bda9a6f22ba436ef9b63b7c086cc80190b (diff) | |
| download | qemu-74de807f794ac5201b2b3c38ddadeef84a676a97.tar.gz qemu-74de807f794ac5201b2b3c38ddadeef84a676a97.tar.bz2 qemu-74de807f794ac5201b2b3c38ddadeef84a676a97.zip | |
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>
| -rw-r--r-- | target-alpha/translate.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/target-alpha/translate.c b/target-alpha/translate.c index 87950c63ec..9909c70b1b 100644 --- a/target-alpha/translate.c +++ b/target-alpha/translate.c @@ -2916,7 +2916,7 @@ void gen_intermediate_code(CPUAlphaState *env, struct TranslationBlock *tb) 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 |