ecpg: Fix rare memory leaks

petere · petere · commit b108a77505f5 · 2012-03-11T01:01:48.000+02:00
found by Coverity
diff --git a/src/interfaces/ecpg/ecpglib/execute.c b/src/interfaces/ecpg/ecpglib/execute.c
@@ -1777,6 +1777,7 @@ ECPGdo(const int lineno, const int compat, const int force_indicator, const char
 		{
 			setlocale(LC_NUMERIC, oldlocale);
 			ecpg_free(oldlocale);
+			free_statement(stmt);
 			va_end(args);
 			return (false);
 		}
@@ -1808,6 +1809,7 @@ ECPGdo(const int lineno, const int compat, const int force_indicator, const char
 			ecpg_raise(lineno, ECPG_INVALID_STMT, ECPG_SQLSTATE_INVALID_SQL_STATEMENT_NAME, stmt->command);
 			setlocale(LC_NUMERIC, oldlocale);
 			ecpg_free(oldlocale);
+			free_statement(stmt);
 			va_end(args);
 			return (false);
 		}

Original file line number	Diff line number	Diff line change
`@@ -1777,6 +1777,7 @@ ECPGdo(const int lineno, const int compat, const int force_indicator, const char`
`1777`	`1777`	`{`
`1778`	`1778`	`setlocale(LC_NUMERIC, oldlocale);`
`1779`	`1779`	`ecpg_free(oldlocale);`
	`1780`	`+ free_statement(stmt);`
`1780`	`1781`	`va_end(args);`
`1781`	`1782`	`return (false);`
`1782`	`1783`	`}`
`@@ -1808,6 +1809,7 @@ ECPGdo(const int lineno, const int compat, const int force_indicator, const char`
`1808`	`1809`	`ecpg_raise(lineno, ECPG_INVALID_STMT, ECPG_SQLSTATE_INVALID_SQL_STATEMENT_NAME, stmt->command);`
`1809`	`1810`	`setlocale(LC_NUMERIC, oldlocale);`
`1810`	`1811`	`ecpg_free(oldlocale);`
	`1812`	`+ free_statement(stmt);`
`1811`	`1813`	`va_end(args);`
`1812`	`1814`	`return (false);`
`1813`	`1815`	`}`