Console View

Categories: connectors experimental galera main

Legend:

Passed

Failed

Warnings

Failed Again

Running

Exception

Offline

No data

connectors

experimental

galera

main

fa80bd28ab91...

fa80bd28ab91aa765811d240625e7b5614525a64

Alexey (Holyfoot) Botchkov

holyfoot@mariadb.com

MDEV-40394 XML schema fails on self-referencing type.

Copy XMLSchema_user_type information when the type is used recursively.
Copies that were once created are stored in m_c_free and then reused
later to avoid memory issues.

f7a90e81d240...

f7a90e81d240cc21244aa73cdb842ad9ca9bccfe

Brandon Nesterenko

brandon.nesterenko@mariadb.com

MDEV-38796 Replace mysql_bin_log.is_open with binlog_ready()

Addresses TODO

- Remove all testing of mysql_bin_log.is_open(). Instead test for
binlog_ready() in main code and add testing of is_open() when
trying to commit the binary log. This is needed as currently
mysql_bin_log.is_open() is tested without a mutex which makes
it unreliable.

And also adds tests for the binlog closing for various types of
statements in-between a thd->binlog_ready() check and the commit-time
acquiring of mysql_bin_log.LOCK_log.

Still TODO from the above request:
* WSREP related mysql_bin_log.is_open() changes as a part of MDEV-38865

f277eefa4248...

f277eefa424850a8cba5c2371406047f6395e8bb

Sergei Golubchik

serg@mariadb.org

MDEV-40362 #mysql50# mixes up with table names in the table cache and on disk

1.
Only allow #mysql50# prefix on table names that
can not be decoded from a file name to a table name.
It is an error to put this prefix on a name that can be accessed
as a valid table name without a prefix. Use `t1` not `#mysql50#t1`,
use `a-b` not `#mysql50#a@002db`.

2.
Don't allow creation of new tables with the #mysql50# prefix.

Except when in SQLCOM_ALTER_DB_UPGRADE - main.upgrade test runs
ALTER DATABASE `#mysql50#mysqltest-1` UPGRADE DATA DIRECTORY NAME;
This command moves tables from the old db name to the new db name,
and some of these tables also have old #mysql50# names.

3.
Also don't allow to create such databases, views, partitions,
subpartitions, sequences, triggers.

f221495d590d...

f221495d590d1a90392335333811446e0f3490b8

Sergei Golubchik

serg@mariadb.org

MDEV-40409 ST_GeomFromText stack overrun on deeply nested GeometryCollection

f2129283c1d4...

f2129283c1d4aa2ff09dbe8ebf86b0e94a686f94

forkfun

alice.sherepa@gmail.com

Merge branch 10.6 into 10.11

ed7f657f077f...

ed7f657f077f096a5a50f08a39e2aae984971209

Brandon Nesterenko

brandon.nesterenko@mariadb.com

MDEV-38796 Test updates for forced row logging

ER_BINLOG_STMT_MODE_AND_ROW_ENGINE is no longer an error, but rather, we
temporarily force ROW logging:

@@ -7259,7 +7262,15 @@ int THD::decide_logging_format(TABLE_LIST
*tables)
wsrep_cs().mode() ==
wsrep::client_state::m_local),1))
{
- my_error((error= ER_BINLOG_STMT_MODE_AND_ROW_ENGINE), MYF(0), "");
+ if (is_current_stmt_binlog_format_stmt())
+ {
+ my_printf_error(ER_BINLOG_STMT_MODE_AND_ROW_ENGINE,
+ "Statement cannot be logged as STATEMENT as at "
+ "least one table is limited to row-based logging."
+ " Switching temporarly to row format",
+ MYF(ME_NOTE));
+ set_current_stmt_binlog_format_row();
+ }
}
Tests are updated that relied on this behavior.

ec517d4b242e...

ec517d4b242e38c5e47a4d2a18e57fc980449386

Brandon Nesterenko

brandon.nesterenko@mariadb.com

MDEV-38796 Update BINLOG_STATE_FILTER at DB change time

Addresses TODO

- Update binlog_state with BINLOG_STATE_FILTER only when database or filter
is changed. This means we do not have to call binlog_filter->db_ok() for
every statement in decide_binlog_format().

The option BINLOG_STATE_FILTER only applies to filtering the current
database. The name seems over-reaching, and would likely lead to
confusion. Renamed to BINLOG_STATE_FILTER_DB to be clear the option only
applies to the database.

dbd60d0ad8da...

dbd60d0ad8daa2d01050346aa50b7b102490e456

Sergei Golubchik

serg@mariadb.org

MDEV-40470 GRANT PROXY with empty password incorrectly checks grantor's privileges

empty password and empty plugin name don't mean "authentication
was not specified", they mean "empty authentication was specified"

ce0b7254d52a...

ce0b7254d52a596b6b7e560587b0ec561bb02ae4

Sergei Golubchik

serg@mariadb.org

MDEV-40311 also fix mariadb-show and mariadb-convert-table-format

bc0913ee2265...

bc0913ee2265ee93821ca2ba92e10fb1068edd10

Sergei Golubchik

serg@mariadb.org

MDEV-40395 MyISAM/Aria silently truncate extensions from filenames, if too long

* move all common fn_format flags that mi_create/maria_create use
into one single #define.
* simplify (MY_APPEND_EXT is redundant and MY_REPLACE_EXT does nothing
when no . is present in the filename)
* use MY_SAFE_PATH everywhere (return NULL if too long)
* set my_errno= ENAMETOOLONG on too long
* abort mi_create/maria_create if returned NULL

b55f5649289b...

b55f5649289b3feb0f8b0ea99448e22f28e0cf75

Sergei Golubchik

serg@mariadb.org

cleanup: silence (wrong) maybe-uninitialized warning

gcc 15.2.1 with -Og:

filamdbf.cpp:355:33: error: 'tfp' may be used uninitialized [-Werror=maybe-uninitialized]
355 | } else if (fread(tfp, HEADLEN, 1, infile) != 1) {
| ~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~

a9983bfbb07a...

a9983bfbb07a00c6a8968168fa896ccbdc4ec945

bsrikanth-mariadb

srikanth.bondalapati@mariadb.com

MDEV-40388: sequence.simple fails on replay

When the recording is enabled for the query,
explain select * from seq_15_to_1_step_12345;
it recorded the table context having a DDL definition as: -

CREATE TABLE `seq_15_to_1_step_12345` (
-> `seq` bigint(20) unsigned NOT NULL,
-> PRIMARY KEY (`seq`)
-> ) ENGINE=SEQUENCE DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_uca1400_ai_ci;

Now, when that context is replayed, the DDL statement is executed.
But, we cannot create such the table and instead it errors out saying
ERROR 1050 (42S01): Table 'seq_15_to_1_step_12345' already exists.

The problem is that,
sequence table was determined using tbl->table->s->sequence. The
intention was to not record sequence table information. However, the
check was only partially correct.

Instead, we need an enhanced condition like: -
if (!tbl->is_view() && tbl->table)
{
if (tbl->table->s->sequence ||
lex_string_eq(hton_name(tbl->table->s->db_type()),
STRING_WITH_LEN("SEQUENCE")))
{
continue;
}
}

This makes both the following kinds of queries on sequences, to not
record any table contexts on them: -
1. explain select * from seq_15_to_1_step_12345;
2. create sequence s1;
EXPLAIN select * from s1;

9cff8aa0242d...

9cff8aa0242dddcae4cc0fc967ec1fe738a30ab0

Georg Richter

georg@mariadb.com

Fix copy/paste error, introduced by rev. a7072b5bf0907c38bacf928ea3ee839314e379c6

9841901f46f4...

9841901f46f4c6879e8308d92fb5aa584caa3b18

forkfun

alice.sherepa@gmail.com

Merge branch 10.6 into 10.11

93078fb753e8...

93078fb753e8a4c34e18d305abd7002a65772695

Sergei Golubchik

serg@mariadb.org

MDEV-40358 symlinked table name is not encoded

encode table name in file name in DATA/INDEX DIRECTORY

8ee69168a8a5...

8ee69168a8a5c72a2859cbdfef4d638f998b5622

Sergei Golubchik

serg@mariadb.org

8a642bc8103a...

8a642bc8103aadc6b2087d781ba8e1595616a2c4

Sergei Golubchik

serg@mariadb.org

MDEV-40484 ambiguous view frm parsing when username includes \n

escape user and host names in view frm

in fact, there is no reason to write any field as is, unescaped,
so let's remove FILE_OPTIONS_STRING in the next release

7fe6738f1698...

7fe6738f169851c4f31eb21e67b6206ad8683e73

Brandon Nesterenko

brandon.nesterenko@mariadb.com

MDEV-38796 Remove *_binlog_local_stmt_filter()

TODO Addressed:
- Remove clear/reset xxx_binlog_local_stmt_filter() as this is now handled
by binlog_state.

728a4e7b5d4c...

728a4e7b5d4c943b7e0b92f94068784c368da91d

ParadoxV5

paradox.ver5@gmail.com

MDEV-40298 Use-After-Free when SQL Thread stops in the middle of SHOW SLAVE STATUS

MDEV-36287 acknowledged that SHOW SLAVE STATUS needed to acquire a mutex
lock before accessing the SQL Thread’s THD, as otherwise it may access
invalid memory if a concurrent STOP SLAVE deletes the THD.
But it missed that the SQL Thread’s mutex is `mi->rli.run_lock`,
not `mi->run_lock`, so the bug was still not fixed.

This commit fills in the oversight by acquiring the correct
corresponding lock for each of `Slave_IO_State` & `Slave_SQL_State`.

Reviewed-by: Kristian Nielsen <[email protected]>

6eaaa1fdad43...

6eaaa1fdad43005c7fbe2f825ecc20c80c4cebd5

forkfun

alice.sherepa@gmail.com

Merge branch 10.6 into 10.11

5e479e1be778...

5e479e1be778348d779decaaa8a197c27e7cbce0

Rucha Deodhar

rucha.deodhar@mariadb.com

MDEV-17943: Unfortunate error code/message upon SET DEFAULT ROLE FOR

Analysis:
Wrong error code

Fix:
Used the correct error code

5c04ca552438...

5c04ca5524385356ff7cf84974a7d491974c1f85

Sergei Golubchik

serg@mariadb.org

MDEV-40329 Binary_string::qs_append(char): Assertion `str_length + (uint32) (1) <= Alloced_length' failed in AsText

Geometry::as_wkt() didn't take into account bytes written into the
result string in the recusrive call of get_data_as_wkt().
Gis_geometry_collection::get_data_as_wkt() didn't reserve space at all.
Thus nested geometry collections could overflow the buffer.

583d2fdfabb4...

583d2fdfabb4de4ea986783eeb2bad94bf834f74

Rucha Deodhar

rucha.deodhar@mariadb.com

MDEV-31477: Inconsistent handling while fetching values in json

Analysis:
When we get value in json_value(), if any error occurs while scanning the
json, it is not reported. Only true is returned.
Fix:
Report error along with returning true (error)

581562f94a83...

581562f94a83f29dcf2c6cc761b49ad55d9c287a

Jan Lindström

jan.lindstrom@mariadb.com

MDEV-40056 - Galera Dynamic Variables Susceptible to RCEs

Make check stronger and require OpenSSL version > 1.1.1
for test case.

5617fffc25aa...

5617fffc25aa29e0a48ee959a658716fd20f44ad

forkfun

alice.sherepa@gmail.com

Merge branch 10.6 into 10.11

53c847f7da66...

53c847f7da666d3273303a191cdafa4dc739140a

Brandon Nesterenko

brandon.nesterenko@mariadb.com

MDEV-38796 Remove silent option from mysql_create_db_internal

TODO addressed
- Remove 'silent' option from mysql_create_db_internal and instead use
tmp_disable_binlog() / reenable_binlog()

49bcb6e3e9c2...

49bcb6e3e9c2c4ea43327f9fc02f728685e07da2

Georg Richter

georg@mariadb.com

Fix maxcolumns test

42ac9df02c92...

42ac9df02c92a65f2586e128f04da9a5b56727a0

ParadoxV5

paradox.ver5@gmail.com

423acf79a268...

423acf79a2680965295a3998b0654bb6ac7a16fc

Brandon Nesterenko

brandon.nesterenko@mariadb.com

MDEV-38796: Fix Windows compiler error

40c5d537933a...

40c5d537933abccdedf943a98a47864d7688d5a9

Sergei Golubchik

serg@mariadb.org

MDEV-30942 MSAN_OPTIONS=poison_in_dtor=1 causes failures in free_root()

Materialized_cursor included MEM_ROOT where it itself (Materialized_cursor)
was allocated. Materialized_cursor inherits from Server_side_cursor,
and Server_side_cursor::operator delete frees this mem_root.

This offends MSAN with poison_in_dtor, because when Materialized_cursor
is destroyed, MEM_ROOT is poisoned and Server_side_cursor::operator delete
accesses memory it really shouldn't.

Fix:
* MEM_ROOT is moved into Server_side_cursor. Materialized_cursor is
the only class inheriting from it, so nobody is affected.
* operator delete is replaced with a function cdestroy() which
saves the MEM_ROOT, deletes the object, then frees the root.
* cannot close() from ~Materialized_cursor anymore, main_mem_root
should be copied before the destruction

2f7feb0bbb36...

2f7feb0bbb36b65b007e15682d29d055655569b8

rusher

diego.dupin@mariadb.com

[CONC-786] add infer to CI

2bf355e361db...

2bf355e361db85153fdfba7b8a0f0d114d6a26ce

Sergei Golubchik

serg@mariadb.org

MDEV-40328 ASAN error on malformed WKB multipolygon

before `data+= WKB_HEADER_SIZE` verify that there's enough data.

rewrite this line in Gis_multi_line_string::get_mbr to make clear
that it does not need this check.

186b3b0823d2...

186b3b0823d22b88ec3b56c838312843243c29ac

Rucha Deodhar

rucha.deodhar@mariadb.com

MDEV-25099: Inconsistent error upon updating read-only table with
ROW_FORMAT=COMPRESSED

Analysis:

Warning level is incorrect
Fix:
Changed to correct warning level

157ce25fc7ea...

157ce25fc7ea1345b5216809c4b47701365e5844

ParadoxV5

paradox.ver5@gmail.com