ODBC-390 Using SQL_ATTR_QUERY_TIMEOUT leaks memory

The string with new query text in MADB_AddQueryTime was allocated and then duplicated, when copied to the Stmt structures, but not freed.
mariadb-corporation · Apr 28, 2023 · c0d989b · c0d989b
1 parent 313102f
commit c0d989b
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/ma_statement.c b/ma_statement.c
@@ -662,7 +662,8 @@ void MADB_AddQueryTime(MADB_QUERY* Query, unsigned long long Timeout)
 
   Query->Length= _snprintf(NewStr, NewSize, "SET STATEMENT max_statement_time=%llu FOR %s", Timeout, Query->Original);
 
-  MADB_RESET(Query->Original, NewStr);
+  MADB_FREE(Query->Original);
+  Query->Original= NewStr;
 }
 
 /* {{{ MADB_StmtPrepare */