From: Paolo Bonzini Date: Mon, 19 Oct 2015 14:08:38 +0000 (+0200) Subject: target-alpha: fix uninitialized variable X-Git-Tag: v2.5.0-rc0~25^2~4 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=74de807f794ac5201b2b3c38ddadeef84a676a97;p=thirdparty%2Fqemu.git 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 Signed-off-by: Michael Tokarev --- diff --git a/target-alpha/translate.c b/target-alpha/translate.c index 87950c63ecd..9909c70b1bb 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