[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[qemu-xen staging] target/hppa: Free some temps in do_sub



commit 79826f99feb7222b7804058f0b4ace9ee0546361
Author:     Richard Henderson <richard.henderson@xxxxxxxxxx>
AuthorDate: Mon Jul 20 10:35:00 2020 -0700
Commit:     Richard Henderson <richard.henderson@xxxxxxxxxx>
CommitDate: Fri Jul 24 14:28:33 2020 -0700

    target/hppa: Free some temps in do_sub
    
    Two temps allocated but not freed.  Do enough subtractions
    within a single TB and one can run out of temps entirely.
    
    Fixes: b2167459ae ("target-hppa: Implement basic arithmetic")
    Buglink: https://bugs.launchpad.net/qemu/+bug/1880287
    Tested-by: Sven Schnelle <svens@xxxxxxxxxxxxxx>
    Reviewed-by: Philippe Mathieu-Daudé <f4bug@xxxxxxxxx>
    Signed-off-by: Richard Henderson <richard.henderson@xxxxxxxxxx>
    Message-Id: <20200720174039.517902-1-richard.henderson@xxxxxxxxxx>
---
 target/hppa/translate.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/target/hppa/translate.c b/target/hppa/translate.c
index 52d7bea1ea..4bd22d4820 100644
--- a/target/hppa/translate.c
+++ b/target/hppa/translate.c
@@ -1294,6 +1294,8 @@ static void do_sub(DisasContext *ctx, unsigned rt, 
TCGv_reg in1,
     save_or_nullify(ctx, cpu_psw_cb_msb, cb_msb);
     save_gpr(ctx, rt, dest);
     tcg_temp_free(dest);
+    tcg_temp_free(cb);
+    tcg_temp_free(cb_msb);
 
     /* Install the new nullification.  */
     cond_free(&ctx->null_cond);
--
generated by git-patchbot for /home/xen/git/qemu-xen.git#staging



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.