From: Ivo Raisr <ivosh@ivosh.net>
Date: Fri, 8 Apr 2016 21:29:15 +0000 (+0000)
Subject: Follow up to bug 345307.
X-Git-Tag: svn/VALGRIND_3_12_0~169
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=7cd8f83b968be9c3c0b19b87d4a4727581f137a1;p=thirdparty%2Fvalgrind.git

Follow up to bug 345307.
Passing parameter on x86 architecture was wrong in final_tidyup().


git-svn-id: svn://svn.valgrind.org/valgrind/trunk@15854
---

diff --git a/coregrind/m_main.c b/coregrind/m_main.c
index 8cfaf820e7..80f347e6c1 100644
--- a/coregrind/m_main.c
+++ b/coregrind/m_main.c
@@ -2819,9 +2819,9 @@ static void final_tidyup(ThreadId tid)
    /* Pass a parameter to freeres_wrapper(). */
 #  if defined(VGA_x86)
    Addr sp = VG_(threads)[tid].arch.vex.guest_ESP;
-   sp = sp - sizeof(UWord);
    *((UWord *) sp) = to_run;
    VG_TRACK(post_mem_write, Vg_CoreClientReq, tid, sp, sizeof(UWord));
+   sp = sp - sizeof(UWord);
    VG_(threads)[tid].arch.vex.guest_ESP = sp;
    VG_TRACK(post_reg_write, Vg_CoreClientReq, tid,
             offsetof(VexGuestX86State, guest_ESP),