12 Commits (feature/pgsql-native-backend-protocol)

Author	SHA1	Message	Date
Rene Cannao	b99e8c57ed	fix(test/tap): wrong handle/column bugs + leaks in two more tests ... test_connection_annotation-t.cpp: - Fix wrong-handle diagnostic: after mysql_real_connect(proxysql_admin) failed, the error path reported mysql_error(proxysql_mysql) — the handle that had already connected successfully. Reports the failed admin handle now. - Fix atoi(row[2]) reading 'srv_host' (a string) as srv_port: the SELECT column order is ConnUsed, ConnFree, srv_host, srv_port, so row[3] is the port. The diagnostic was reporting garbage values like 127 (for host "127.0.0.1") as the port. - Add mysql_init NULL checks for both proxysql_mysql and proxysql_admin; fix cross-leaks when one succeeds and the other fails. - Fix proxysql_mysql/proxysql_admin leaks on all real_connect and store_result error paths, plus at happy-path exit. - NULL-check both mysql_store_result calls before passing the result to mysql_fetch_row (was UB on NULL). - Guard the vector subtraction loop against mismatched row counts using std::min of both vector sizes; previously a shorter 'new_ cur_connections' would be read out of bounds. mysql-fast_forward-t.cpp: - Same leak pattern as kill_connection{2,3}-t.cpp: NULL-deref UB on mysql_init failure, create_connections leaking the in-flight handle on real_connect failure, and the entire conns[NUM_CONNS] + admin handle leaked on every exit path. - Between the two create_connections calls, add explicit close_all_conns() so the second batch doesn't orphan the first. - Add NULL check on mysql_store_result(conns[0]) before passing to mysql_fetch_row in the SELECT * FROM test.sbtest1 loop. Compile-verified with -fsyntax-only.	3 months ago
Rene Cannao	12dc2777b9	test: Fix test_connection_annotation-t by adding warmup and forcing HG1	4 months ago
Miro Stauder	803510fa5e	fix mysql includes	3 years ago
Miro Stauder	c78819bda9	use inludes from our deps intead of system	3 years ago
Miro Stauder	f865bb504e	fix connection strings and related	3 years ago
René Cannaò	25da524c41	Adding plan() in several TAP tests	3 years ago
Javier Jaramago Fernández	bcb009fc24	Revert "Merge pull request #3645 from sysown/v2.x-2021" ... This reverts commit 9561cb3ef2, reversing changes made to 7edfec6f3f.	5 years ago
René Cannaò	a6edd02a9b	Revert "Merge branch 'v2.x' into v2.x-2021" ... This reverts commit 3a3931f2d4, reversing changes made to 4694ab775e.	5 years ago
Javier Jaramago Fernández	ec8f19a82d	Removed unnecessary '+' mark from query annotations in tap tests	5 years ago
Javier Jaramago Fernández	d9a0e26feb	Added extra logging reporting 'ConnUsed' and 'ConnFree' for each server	6 years ago
Javier Jaramago Fernández	cdf4169572	Test 'test_connection_annotation' now also takes into account 'ConnUsed' when counting new created connections	6 years ago
Javier Jaramago Fernández	28e4a59b0b	Added test for new annotation 'create_new_connection'	6 years ago