From: Aleksey Demakov <ademakov@gmail.com>
Date: Sat, 14 Oct 2006 04:24:31 +0000 (+0000)
Subject: fix uninitialized value use in create_value_entry();
X-Git-Tag: before.move.to.git~186
X-Git-Url: https://git.unchartedbackwaters.co.uk/w/?a=commitdiff_plain;h=5f1a39f6b1dedecab5a2979cfdcf329fc9671087;p=francis%2Flibjit.git

fix uninitialized value use in create_value_entry();
fix return code in interpreter's version of _jit_create_call_return_insns().
---

diff --git a/ChangeLog b/ChangeLog
index df3c1af..d2408be 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2006-10-14  Aleksey Demakov  <ademakov@gmail.com>
+
+	* jit/jit-rules-interp.c (_jit_create_call_return_insns): fix return
+	code in case of no return value or struct returned via pointer (this
+	is just like 2006-02-20 fix for x86).
+
+	* jit/jit-cfg.c (create_value_entry): an uninitialized value was
+	used (thanks Klaus for spotting).
+
 2006-10-02  Klaus Treichel  <ktreichel@web.de>
 
 	* jit/jit-intrinsic.c: Fix a copy bug in jit_ulong_mul (replace - by *).
diff --git a/jit/jit-cfg.c b/jit/jit-cfg.c
index cec465d..0f2c1d7 100644
--- a/jit/jit-cfg.c
+++ b/jit/jit-cfg.c
@@ -467,7 +467,7 @@ create_value_entry(_jit_cfg_t cfg, jit_value_t value)
 		}
 		else
 		{
-			max_values += max_values / 2;
+			max_values = cfg->max_values * 2;
 			values = jit_realloc(cfg->values, max_values * sizeof(struct _jit_value_entry));
 		}
 		if(!values)
diff --git a/jit/jit-rules-interp.c b/jit/jit-rules-interp.c
index e38cebe..879ad9e 100644
--- a/jit/jit-rules-interp.c
+++ b/jit/jit-rules-interp.c
@@ -592,7 +592,7 @@ int _jit_create_call_return_insns
 	/* Bail out now if we don't need to worry about return values */
 	if(!return_value || ptr_return)
 	{
-		return 0;
+		return 1;
 	}
 
 	/* Structure values must be flushed into the frame, and