mariadb

Tree: 8258b2fd56

Author	SHA1	Message	Date
Alexey Botchkov	fbec528cbb	MDEV-36245 review changes Closes #3874	8 months ago
Luis Contreras	35c25cd107	MDEV-36412 Concerns compilation issue on community edition for x86_64 with X32 ABI	7 months ago
Monty	51c5b75335	Always call mysql_cond_broadcast(&rli->data_cond) under data_lock This is a safetly fix to try to fix random failures in parallel_backup_xa_debug reported as: sync_slave_with_master failed: 'select master_pos_wait('master-bin.000001', 1034, 300, '')' returned -1 One possible reason could be lost signals, which this patch fixes.	7 months ago
Tony Chen	8c6b0d092a	MDEV-36245 Long server_audit_file_path causes buffer overflow Limit size of server_audit_file_path value Currently, the length of this value is not checked and can cause a buffer overflow if given a long file path specifying a directory. In file_logger:logger_open(), there is a check: ``` if (new_log.path_len+n_dig(rotations)+1 > FN_REFLEN) // handle error ``` As n_dig(rotations) may return up to 3, this inherently limits the file path to FN_REFLEN - 4 characters. All new code of the whole pull request, including one or several files that are either new files or modified ones, are contributed under the BSD-new license. I am contributing on behalf of my employer Amazon Web Services, Inc.	8 months ago
ParadoxV5	5b1bdf6076	MDEV-36359: Patch NULL deref after disabling Semi-Sync primary Under `@@rpl_semi_sync_master_wait_no_slave = 0`, when `rpl_semi_sync_master_clients` decrements to zero, the primary reverts to async replication. This code did not check whether Semi- Sync is still globally enabled or not as it didn’t matter before. However, after MDEV-33551 (#3089) split the transactions’ACK condition variables to per-transaction, this function now needs Semi-Sync’s transaction tracker to unblock these condition variables in batch, but this tracker is `NULL` when Semi-Sync Primary is disabled. Co-authored-by: Kristian Nielsen <knielsen@knielsen-hq.org>	6 months ago
Sergei Golubchik	7f1492d0bc	cleanup: rename hide_view_error->replace_view_error_with_generic as requested by Monty	6 months ago
Sergei Golubchik	f99586668a	MDEV-36380 User has unauthorized access to a sequence through a view with security invoker check sequence privileges in Item_func_nextval::fix_fields(), just like column privileges are checked in Item_field::fix_fields() remove sequence specific hacks that kinda made sequence privilege checks works, but not in all cases. And they were too lax, didn't requre SELECT privilege for NEXTVAL. Also INSERT privilege looks wrong here, UPDATE would've been more appropriate, but won't change that for compatibility reasons. also fixes MDEV-36413 User without any privileges to a sequence can read from it and modify it via column default	6 months ago
Daniel Black	839e8bfe9f	MDEV-36182: liburing - incorrect error handing. As noted by Jens Axobe, errno isn't set, it is returned by the io_uring_queue_init function. As such users where getting the following as the common case when EPERM was actually intended to be returned. "mariadbd: io_uring_queue_init() failed with errno 0". Add to error message the correct relevant information around EPERM.	6 months ago
Yuchen Pei	f89f8aa313	MDEV-36357 MDEV-35452 Temporarily disable view protocol for a query in spider/bg.basic_sql It should be re-enabled once MDEV-36357 is fixed. Also added some documentation to spider result operations.	6 months ago
Daniel Black	73cdeda347	tpool: remove m_group_enqueued (unused) Noted in MDEV-36542 that it wasn't used and the compile warning -Wunused-private-field is legitimate. Review: Vladislav Vaintroub	6 months ago
Daniel Black	1055bc957e	MDEV-36182: Revert "MDEV-26674: Set innodb_use_native_aio=OFF ...when using io_uring on a potentially affected kernel" Remove version check on the kernel as it now corresponds to a working RHEL9 kernel and the problem was only there in pre-release kernels that shouldn't have been used in production. This reverts commit `1193a793c4`.	6 months ago
Daniel Black	51179067fc	MDEV-36182: Revert "MDEV-26674 workaround for mariadb-backup" Remove version check on the kernel as it now corresponds to a working RHEL9 kernel and the problem was only there in pre-release kernels that shouldn't have been used in production. This reverts commit `3dc0d884ec`.	6 months ago
Marko Mäkelä	1c9f64e54f	MDEV-36613 Incorrect undo logging for indexes on virtual columns Starting with mysql/mysql-server@02f8eaa9988dadb73dd68630dd82393cfa19bfb8 and commit `2e814d4702` the index ID of indexes on virtual columns was being encoded insufficiently in InnoDB undo log records. Only the least significant 32 bits were being written. This could lead to some corruption of the affected indexes on ROLLBACK, as well as to missed chances to remove some history from such indexes when purging the history of committed transactions that included DELETE or an UPDATE in the indexes. dict_hdr_create(): In debug instrumented builds, initialize the DICT_HDR_INDEX_ID close to the 32-bit barrier, instead of initializing it to DICT_HDR_FIRST_ID (10). This will allow the changed code to be exercised while running ./mtr --suite=gcol,vcol. trx_undo_log_v_idx(): Encode large index->id in a similar way as mysql/mysql-server@e00328b4d068c7485ac2ffe27207ed1f462c718d but using a different implementation. trx_undo_read_v_idx_low(): Decode large index->id in a similar way as mach_u64_read_much_compressed(). Reviewed by: Debarun Banerjee	6 months ago
Jan Lindström	5f2562291c	MDEV-36509 : Galera test failure on galera_sr.mysql-wsrep-features#165 Problem was that thread was holding lock_sys.wait_mutex when streaming replication transaction rollback was handled and in wsrep-lib requests THD::LOCK_thd_kill mutex causing wrong mutex usage (thd->reset_globals()). Fix is to remove streaming replication rollback handling from Deadlock::report() i.e. wsrep_handle_SR_rollback call. Purpose of Deadloc::report() is to find a cycle in the waits-for graph if exists, report it, mark victim transaction as deadlock victim and release locks it is waiting for. Actual streaming replication rollback that can take longer time can be handled later at trx_t::rollback where lock_sys.wait_mutex is not held. Signed-off-by: Julius Goryavsky <julius.goryavsky@mariadb.com>	7 months ago
Jan Lindström	ee947fae80	MDEV-36464 : Galera test failure on galera_3nodes.galera_gtid_2_cluster Test changes only. Add wait conditions after INSERT-clauses to make sure that they are replicated before checking gtid position or table contents. Signed-off-by: Julius Goryavsky <julius.goryavsky@mariadb.com>	7 months ago
Julius Goryavsky	0403f0147f	MDEV-33136: backport corrections from 10.11+	7 months ago
Julius Goryavsky	88dfa6bcee	Merge branch '10.5' into '10.6'	6 months ago
Julius Goryavsky	08f902e9ce	MDEV-36116: correction for error codes	7 months ago
Alexey Yurchenko	77391482bd	MDEV-34998: master can stop responding after cluster vote to evict a node Additional corrections: there is a natural race between closing connection to cluster in case of applying error and finishing the IST and sometimes IST finishes and tries to send JOIN message over a closed connection. This does not affect the correctness of the test or node behavior. Added error message suppression. Signed-off-by: Julius Goryavsky <julius.goryavsky@mariadb.com>	6 months ago
Julius Goryavsky	bbd0e4b2c9	MDEV-34998 addendum: post-fix corrections for SST scripts Delaying scripts on joiner after SST/IST has been made a common debug feature for all suitable SST/IST methods. Also some minor fixes have been made for new tests.	7 months ago
Alexey Yurchenko	ec5068fe59	MDEV-34998: master can stop responding after cluster vote to evict a node After cluster vote to evict a node that failed a transaction, current master can't commit anymore. Error voting for joiner in the JOINED state was broken because the group-wide commit cut (implicit SUCCESS vote) was not taken into account when processing error vote request from the JOINED node. This commit adds 3 MTR tests to verify the fix in the galera library works as designed. Requires Galera library commit `91f0090a05` (galera library version > 26.4.19). Signed-off-by: Julius Goryavsky <julius.goryavsky@mariadb.com>	1 year ago
Nikita Malyavin	cb7e39b75b	MDEV-36181 Field pointer may be uninitialized in fill_record Newer gcc reports: error: 'rfield' may be used uninitialized [-Werror=maybe-uninitialized] 9041 \| unwind_stored_field_offsets(fields, rfield); After investigation, it turned to be an impossible case: 1. The only way it could be broken is if if (!(field= fld->field_for_view_update())) line case would succeed from the first time. 2. Consequent checks initialize rfield. fld may return NULL in field_for_view_update() only for views. 3. Before fill_record, UPDATE first calls check_fields, where field_for_view_update() result is already checked. INSERT calls check_view_insertability that checks that all view fields are updateable. It all means that field_for_view_update() cannot be NULL in fill_record, so the if can be converted to DBUG_ASSERT. This essentially shifts the responsibility on preliminary field_for_view_update() check to the caller. In this patch: 1. convert field_for_view_update() check to DBUG_ASSERT 2. harden unwind_stored_field_offsets function so that it can be used even if field_for_view_update() is NULL 3. As a consequence, `field` is passed instead of `rfield` as a terminator. 4. Initialize `field` to NULL to bypass a false-positive warning!	7 months ago
Oleksandr Byelkin	ba34657cd2	MDEV-35238 (MDEV-34922) Wrong results from a tables with a single record and an aggregate The problem is that copy function was used in field list but never copied in this execution path. So copy should be performed before returning result. Protection against uninitialized copy usage added.	9 months ago
Marko Mäkelä	14389b6993	MDEV-36510 InnoDB fails to compile with clang++-20 ut_allocator(PSI_memory_key): Remove the explicit keyword, to allow implicit instantiation of purge_sys_t::unordered_map.	6 months ago
Nikita Malyavin	e6ea5d568c	MDEV-36507 fix dbug_print_row concurrent access `7544fd4cae` had to make use of a static array to avoid memory use-after-free or leak. Instead, let us make a function returning String, this is the only way to automatically manage the memory after the function returned. To make it all correct, move constructor is added. Normally, it is expected, that the constructor will be elided upon return of an object by value, but if something goes different, or -fno-elide-constructors is used, we can have a problem. So this was a move constructor avoids copy elision-related UB. dbug_print_row returning char* is still there for convenient use in a debugger.	8 months ago
Nikita Malyavin	4de5161a70	fix build with -fno-elide-constructors Linker is trying to find a copy constructor to injector::transaction::transaction ld.lld: error: undefined symbol: injector::transaction::transaction (injector::transaction const&) >>> referenced by rpl_injector.cc:164 >>> rpl_injector.cc.o:(injector::new_trans(THD*)) This constructor is declared, but is not implemented. Ok if copy elision is enabled, but causes error otherwise. Remove the constructor declaration, so that operator= will be used in reality.	7 months ago
Nikita Malyavin	24a4d4ab0c	MDEV-36422 Build fails with cmake 4.0.0 due to wsrep update wsrep-lib	7 months ago
Nikita Malyavin	2a5a12b227	MDEV-36506 Build fails with cmake 4.0 Update cmake_minimum_required to 2.8...3.12 in root cmake and mroonga. This will update "Policy Version" to 3.12, which will not prevent the build by even higher cmake versions. There is also a reason to stay on the compatible with windows "policy version", so 3.12 is conservatively chosen. On the other hand, it will require at least version 2.8.	7 months ago
Yuchen Pei	b6392c292e	MDEV-36454 MDEV-35452 Fix spider view protocol test failures caused by tampering of order by items With view protocol, a SELECT statement is transformed into two statements: 1. CREATE OR REPLACE VIEW mysqltest_tmp_v AS SELECT ... 2. SELECT * FROM mysqltest_tmp_v The first statement reconstructed the query, which is executed in the second statement. The reconstruction often replaces aliases in ORDER BY by the original item. For example, in the test spider/bugfix.mdev_29008 the query SELECT MIN(t2.a) AS f1, t1.b AS f2 FROM tbl_a AS t1 JOIN tbl_a AS t2 GROUP BY f2 ORDER BY f1, f2; is transformed to "select min(`t2`.`a`) AS `f1`,`t1`.`b` AS `f2` from (`auto_test_local`.`tbl_a` `t1` join `auto_test_local`.`tbl_a` `t2`) group by `t1`.`b` order by min(`t2`.`a`),`t1`.`b`" In such cases, spider constructs different queries to execute at the data node. So we disable view protocol for such queries.	7 months ago
Yuchen Pei	8866fba00a	MDEV-36454 MDEV-35452 Fix spider view protocol test failures caused by different queries constructed without GBH With view protocol, often during optimization, the GBH is not created because join->tables_list is the view mysqltest_tmp_v which has MEMORY as engine which does not have GBH implemented. In such cases, if without view protocol the test takes a path that does create a spider GBH, the resulting queries sent to the data node often differ. Therefore we disable view protocol for these statements.	7 months ago
Yuchen Pei	dabd51c391	MDEV-36335 MDEV-35452 Disable view protocol when calling spider_copy_tables in tests Spider needs to lock the spider table when executing the udf, but the server layer would have already locked tables in view protocol because it transforms the query: select spider_copy_table('t', 0, 1) to two queries create or replace view mysqltest_tmp_v as select spider_copy_table('t', 0, 1); select * from mysqltest_tmp_v; So spider justifiably errors out in this case by checking on thd->derived_tables and thd->locks in spider_copy_tables_body()	7 months ago
Yuchen Pei	0f7c9146cf	MDEV-36452 MDEV-35452 Fix udf output temp field name in spider tests with --view-protocol	7 months ago
Yuchen Pei	25f1e6f565	MDEV-36307 MDEV-35452 Do not create spider group by handler when dealing with derived tables or view and at least one select item is constant If one of the selected field is a MIN or MAX and it has been optimized into a constant, it is not added to the temp table used by a group by handler (GBH). The GBH therefore cannot store results to this missing field. On the other hand, when SELECTing from a view or a derived table, TMP_TABLE_ALL_COLUMNS is set. If the query has no group by or order by, an Item_temptable_field is created for this MIN/MAX field and added to the JOIN. Since the GBH could not store results to the corresponding field in the temp table, the value of this Item_temptable_field remains NULL. And the NULL value is passed to the record, then the temp row, and finally output as the (wrong) result. To fix this, we opt to not creating a spider GBH when a view or derived table is involved. This fixes spider/bugfix.mdev_26345 for --view-protocol Also fixed a comment: TABLE_LIST::belong_to_derived is NULL if the table belongs to a derived table that has non-MERGE type.	7 months ago
Yuchen Pei	59962ae2b3	MDEV-36442 MDEV-35452 Fix --view-protocol for spider tests with SELECT arguments of SELECT statements Running mtr --view-protocol transforms SELECT statements to a CREATE OR REPLACE VIEW of the statement, followed by SELECT from the view. When thus when spider tests check the query log for select statements, it often output a different one with --view-protocol compared to without. By adding disable/enable_view_protocol pairs to these statements. Most of these statements are surrounded by existing disable/enable_ps[2]_protocol pairs. Acked-by: Yuchen Pei <ycp@mariadb.com>	7 months ago
Sergey Vojtovich	b02ad4a6f8	MDEV-36427 - FTBFS with libxml2 2.14.0 Connect engine fails to build with libxml2 2.14.0. Connect engine uses "#ifndef BASE_BUFFER_SIZE" to determine if libxml2 is available. If libxml2 is unavailable it did redefine xmlElementType enum of libxml/tree.h. The reasons for this redefinition is vague, most probably some of these constants were used when connect was compiled with MSXML, while libxml2 was disabled. However BASE_BUFFER_SIZE constant was removed from libxml2 recently, as a result connect fails to build due to xmlElementType constants redefinition. Use LIBXML2_SUPPORT instead of BASE_BUFFER_SIZE for libxml2 availability check.	7 months ago
Daniel Black	690b2cf776	test: archive-big test too big for msan archive.archive-big w1 [ fail ] timeout after 900 seconds Test ended at 2025-03-19 22:27:30 Test case timeout after 900 seconds == /build/mysql-test/var/1/log/archive-big.log == CREATE TABLE t1(a BLOB) ENGINE=ARCHIVE; INSERT INTO t1 SELECT * FROM t1; ... == /build/mysql-test/var/1/tmp/analyze-timeout-mysqld.1.err == mysqltest: Could not open connection 'default' after 500 attempts: 2002 Can't connect to local MySQL server through socket '/build/mysql-test/var/tmp/1/mysqld.1.sock' (111)	7 months ago
Daniel Black	93ea4f29a4	MDEV-36347 UBSAN: plugins.auth_v0100 - runtime error: call to function do_auth_0x0100 through pointer to incorrect function type. Redoing a new plugin interface for an obsolete protocol was too much so we just remove the UBSAN testing on the function. It wasn't possible to just disable funtion-type-mismatch	7 months ago
Daniel Black	3995de0318	MDEV-36341 UBSAN: FederatedX fill_server runtime error: applying non-zero offset to null pointer UBSAN is nuanced around null pointer additions, but there is no offset on schema, its the start of the server->key.	7 months ago
Daniel Black	13dd073742	MDEV-31846: enable cursor protocol for test federatedx_create_handlers Per comment it code - it can be removed 10.5+. Not in the comment, is the SELECT INTO ... parts still required cursor protocol exclusion.	7 months ago
Daniel Black	b316a7135b	mroonga: undefined behaviour fix The value of dv[0].data being null showed up in the mtr tests: mroonga/storage.alter_table_fulltext_add_no_primary_key as: /source/storage/mroonga/vendor/groonga/lib/ii.c:2052:37: runtime error: applying non-zero offset 28 to null pointer Correct this by entrying the if condition on null pointer value. The free is valid, and the data of size is allocated.	7 months ago
Daniel Black	2ba49bd804	mroonga: correct offsetof calculation Using a null pointer as a basic for calculation was undefined behaviour.	7 months ago
Marko Mäkelä	db4763a0d1	Fix a slow test When we expect a lock wait timeout, let us override the default innodb_lock_wait_timeout=50 with the minimum timeout of 1 second.	7 months ago
Thirunarayanan Balathandayuthapani	b11772d9a5	MDEV-33167 ASAN errors in dict_sys_t::load_table / get_foreign_key_info after failing to load a table Problem: ======= - While loading the foreign key constraints for the parent table, if child table wasn't open then InnoDB uses the parent table heap to store the child table name in fk_tables list. If the consecutive foreign key relation for the parent table fails with error, InnoDB evicts the parent table from memory. But InnoDB accesses the evicted table memory again in dict_sys.load_table() Solution: ======== dict_load_table_one(): In case of error, remove the child table names which was added during dict_load_foreigns()	7 months ago
Thirunarayanan Balathandayuthapani	0d7ef4f478	MDEV-36236 Instant alter aborts when InnoDB fails to rollback instant operation Problem: ======== - InnoDB does consecutive instant alter operation, first instant DDL fails, it fails to reset the old instant information in table during rollback. This lead to consecutive instant alter to have wrong assumption about the exisitng instant column information. Fix: ==== dict_table_t::instant_column(): Duplicate the instant information field of the table. By doing this, InnoDB alter retains the old instant information and reset it during rollback operation	7 months ago
Marko Mäkelä	e7442e5eb9	MDEV-36226 fixup: format mismatch	7 months ago
Marko Mäkelä	4c0e2f1aca	MDEV-35813: even more robust test case The test in commit `1756b0f37d` is occasionally failing if there are unexpectedly many page cleaner batches that are updating the log checkpoint by small amounts. This occurs in particular when running the server under Valgrind. Let us insert the same number of records with a larger number of statements in a hope that the test would then be more likely to pass.	7 months ago
Denis Protivensky	c01bff4a10	MDEV-36360: Don't grab table-level X locks for applied inserts It prevents a crash in wsrep_report_error() which happened when appliers would run with FK and UK checks disabled and erroneously execute plain inserts as bulk inserts. Moreover, in release builds such a behavior could lead to deadlocks between two applier threads if a thread waiting for a table-level lock was ordered before the lock holder. In that case the lock holder would proceed to commit order and wait forever for the now-blocked other applier thread to commit before. Signed-off-by: Julius Goryavsky <julius.goryavsky@mariadb.com>	7 months ago
Julius Goryavsky	b983a911e9	galera mtr tests: synchronization between branches and editions	7 months ago
Julius Goryavsky	5003dac220	MDEV-36116: Post-merge correction for 10.6+	7 months ago
Julius Goryavsky	03c31ab099	Merge branch '10.5' into '10.6'	7 months ago

... 3 4 5 6 7 ...

199626 Commits (8258b2fd56e6c7fad64d3877ca01049778f1a541) All Branches Search

199626 Commits (8258b2fd56e6c7fad64d3877ca01049778f1a541)

All Branches