In ecpglib restore LC_NUMERIC in case of an error.

Michael Meskes · Michael Meskes · commit f8c74422010e · 2011-09-01T15:31:16.000+02:00
diff --git a/src/interfaces/ecpg/ecpglib/execute.c b/src/interfaces/ecpg/ecpglib/execute.c
@@ -1773,8 +1773,12 @@ ECPGdo(const int lineno, const int compat, const int force_indicator, const char
 	 */
 	if (statement_type == ECPGst_prepnormal)
 	{
-		if (!ecpg_auto_prepare(lineno, connection_name, compat, &prepname, query))
+		if (!ecpg_auto_prepare(lineno, connection_name, compat, &prepname, query)) {
+			setlocale(LC_NUMERIC, oldlocale);
+			ecpg_free(oldlocale);
+			va_end(args);
 			return (false);
+		}
 
 		/*
 		 * statement is now prepared, so instead of the query we have to
@@ -1801,6 +1805,9 @@ ECPGdo(const int lineno, const int compat, const int force_indicator, const char
 		else
 		{
 			ecpg_raise(lineno, ECPG_INVALID_STMT, ECPG_SQLSTATE_INVALID_SQL_STATEMENT_NAME, stmt->command);
+			setlocale(LC_NUMERIC, oldlocale);
+			ecpg_free(oldlocale);
+			va_end(args);
 			return (false);
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -1773,8 +1773,12 @@ ECPGdo(const int lineno, const int compat, const int force_indicator, const char`
`1773`	`1773`	`*/`
`1774`	`1774`	`if (statement_type == ECPGst_prepnormal)`
`1775`	`1775`	`{`
`1776`		`- if (!ecpg_auto_prepare(lineno, connection_name, compat, &prepname, query))`
	`1776`	`+ if (!ecpg_auto_prepare(lineno, connection_name, compat, &prepname, query)) {`
	`1777`	`+ setlocale(LC_NUMERIC, oldlocale);`
	`1778`	`+ ecpg_free(oldlocale);`
	`1779`	`+ va_end(args);`
`1777`	`1780`	`return (false);`
	`1781`	`+ }`
`1778`	`1782`
`1779`	`1783`	`/*`
`1780`	`1784`	`* statement is now prepared, so instead of the query we have to`
`@@ -1801,6 +1805,9 @@ ECPGdo(const int lineno, const int compat, const int force_indicator, const char`
`1801`	`1805`	`else`
`1802`	`1806`	`{`
`1803`	`1807`	`ecpg_raise(lineno, ECPG_INVALID_STMT, ECPG_SQLSTATE_INVALID_SQL_STATEMENT_NAME, stmt->command);`
	`1808`	`+ setlocale(LC_NUMERIC, oldlocale);`
	`1809`	`+ ecpg_free(oldlocale);`
	`1810`	`+ va_end(args);`
`1804`	`1811`	`return (false);`
`1805`	`1812`	`}`
`1806`	`1813`	`}`