Home - Waterfall Grid T-Grid Console Builders Recent Builds Buildslaves Changesources - JSON API - About

Console View


Categories: connectors experimental galera main
Legend:   Passed Failed Warnings Failed Again Running Exception Offline No data

connectors experimental galera main
f9b58f4ce8ff...
f9b58f4ce8ffa058c203a59ed648a75aaeef842d
Aleksey Midenkov
MDEV-25529 ALTER TABLE FORCE syntax improved

Improves ALTER TABLE syntax when alter_list can be supplied alongside a
partitioning expression, so that they can appear in any order. This is
particularly useful for the FORCE clause when adding it to an existing
command.

Also improves handling of AUTO with FORCE, so that AUTO FORCE
specified together provides more consistent syntax, which is used by
this task in further commits.
f5bb99221076...
f5bb9922107672e88f7b5cbdb3d25151cc5744bb
Raghunandan Bhat
MDEV-39271: SIGSEGV in `check_word`|(`extract_date_time`|`extract_oracle_date_time`)

Problem:
  Date and time functions - `STR_TO_DATE` and `TO_DATE` (in Oracle mode)
  both use `check_word` to check if a string token matches a valid
  locale-specific day or month name. Server crashed because of an
  un-initialized typelib member(`type_lengths`) for date locales which
  are not present in `Oracle_date_locale` (e.g: `ar_DZ`).

Fix:
  Initialize `type_lengths` for all supported locales by iterating over
  `my_locales` array instead of `Oracle_date_locale` during server
  startup.
f39b634db715...
f39b634db715cd9dc1835653d1ce544df2aa1613
Vladislav Vaintroub
MDEV-39258 Make THIRD_PARTY_DOWNLOAD_LOCATION settable

Also, on request of Razvan, increase HeidiSQL download timeout
ec651c18adda...
ec651c18adda8847613c5b4a4ccd01b55b5d7396
Michal Schorm
CONC-816: INCLUDE(FindXXX) -> FIND_PACKAGE(XXX)

INCLUDE(FindXXX) directly includes the find module, bypassing
cmake's package search infrastructure. FIND_PACKAGE(XXX) is
the standard form and has been preferred since cmake 2.x.

Co-Authored-By: Claude AI <[email protected]>
e7ecbca69c14...
e7ecbca69c14245949a87a90c41d53e24bb215a8
Michal Schorm
CONC-816: drop redundant conditions from ENDIF/ELSE

Old cmake style repeated the condition in ENDIF(condition)
and ELSE(condition). Modern cmake uses bare ENDIF()/ELSE().

Co-Authored-By: Claude AI <[email protected]>
e1580dd43e9b...
e1580dd43e9b750aa90b70a465a0fa9460c85c26
Alexey Botchkov
MDEV-38767 XML datatype to be reported as format in extended metadata in protocol.

add respective Send_field metadata for UDT.
ddf3fbb3c04f...
ddf3fbb3c04f27b55f16f7df90c15b669cbd0e2d
Aleksey Midenkov
Vanilla: cleanup for vers_set_starts() and starts_clause
daabd5f1fb14...
daabd5f1fb140a8bbae851812cf3cafb1e504219
Michal Schorm
CONC-816: remove dead cmake version check in misc.cmake

The VERSION_LESS "2.8.7" branch is unreachable since
cmake_minimum_required is 3.12. Keep only the ELSE block
which defines MESSAGE1() with deduplication.

Co-Authored-By: Claude AI <[email protected]>
d0e9ef8698bb...
d0e9ef8698bbb5389b8d53509b1e83d9f96eec3b
Raghunandan Bhat
MDEV-39271: SIGSEGV in `check_word`|(`extract_date_time`|`extract_oracle_date_time`)

Problem:
  Date and time functions - `STR_TO_DATE` and `TO_DATE` (in Oracle mode)
  both use `check_word` to check if a string token matches a valid
  locale-specific day or month name. Server crashed because of an
  un-initialized typelib member(`type_lengths`) for date locales which
  are not present in `Oracle_date_locale` (e.g: `ar_DZ`).

Fix:
  Initialize `type_lengths` for all supported locales by iterating over
  `my_locales` array instead of `Oracle_date_locale` during server
  startup.
ca69331705c8...
ca69331705c800fbe1da664de33d500789b24c34
Michal Schorm
CONC-813, CONC-814 Fix UBSan errors

sint4korr (non-x86 path): the expression
`(int16)(A)[3] << 24` causes signed integer overflow
when byte value >= 128, since 128<<24 exceeds INT32_MAX.
Redefine as `(int32) uint4korr(A)` to compute in unsigned
arithmetic, matching the server (my_byteorder.h:137-140).
The next line sint8korr already uses this same pattern:
`(longlong) uint8korr(A)`.

convert_from_long (MYSQL_TYPE_DOUBLE, MYSQL_TYPE_FLOAT):
casting double/float back to ulonglong/longlong for
truncation detection is undefined when the float value
is outside the representable integer range. For example,
`(double)ULONGLONG_MAX` rounds up to 2^64 in IEEE 754
(not exactly representable with 53-bit mantissa), so
`(ulonglong)dbl` overflows. Similarly for LONGLONG_MAX.
Add range checks before the cast using `>=` for upper
bounds (since the double rounds up past the integer max)
and `<` for LONGLONG_MIN (exactly representable as -2^63).
Out-of-range values set the truncation error flag directly,
short-circuiting the dangerous cast via `||` evaluation.
This matches the server pattern in sql/field.cc and
sql/sql_select.cc (double_to_ulonglong macro).

Co-Authored-By: Claude AI <[email protected]>
c8acc35f9d29...
c8acc35f9d29aa7aa9995c7253bfab5f02a1bd7e
Michal Schorm
CONC-816: use NAMELINK_COMPONENT (available since cmake 3.12)

Replace the NAMELINK_SKIP / NAMELINK_ONLY two-INSTALL workaround
with the single-INSTALL NAMELINK_COMPONENT form. The workaround
was needed for CentOS 7's cmake 2.8.12 which is no longer supported.

Co-Authored-By: Claude AI <[email protected]>
c3ee1036a252...
c3ee1036a2527a391f75b019c6bec0b71caaa42c
Aleksey Midenkov
Vanilla: get_next_time() comment
c33d8cd1b273...
c33d8cd1b27349f5ac716011757d5745fd950d0b
Georg Richter
Merge pull request #306 from FaramosCZ/CONC-816

CONC-816: CMake modernization for the 3.12 baseline
bbd6796a6f9e...
bbd6796a6f9ecf39a3e0dbb8e8157cde8f7f857d
Thirunarayanan Balathandayuthapani
MDEV-39261 MariaDB crash on startup in presence of indexed virtual columns

Problem:
========
A single InnoDB purge worker thread can process undo logs from different
tables within the same batch. But get_purge_table(), open_purge_table()
incorrectly assumes that a 1:1 relationship between a purge worker thread
and a table within a single batch. Based on this wrong assumtion,
InnoDB attempts to reuse TABLE objects cached in thd->open_tables for
virtual column computation.

1) Purge worker opens Table A and caches the TABLE pointer in thd->open_tables.
2) Same purge worker moves to Table B in the same batch, get_purge_table()
retrieves the cached pointer for Table A instead of opening Table B.
3) Because innobase::open() is ignored for Table B, the virtual column
template is never initialized.
4) virtual column computation for Table B aborts the server

Solution:
========
The purge coordinator thread now opens both InnoDB dict_table_t* and
MariaDB TABLE* handles during batch preparation in
trx_purge_attach_undo_recs(). These handles are stored in a new
purge_table_ctx_t structure within each purge_node_t->tables map,
keyed by table_id. When worker thread needs TABLE* for virtual column
computation, it fetches the TABLE* from purge_node_t->tables.

purge_table_ctx_t(): Structure to hold dict_table_t*, MDL_ticket* and
TABLE* for each table during batch processing. TABLE* is being opened
only when virtual index exists for the table.

innobase_open_purge_table(), innobase_close_thread_table(): wrapper to
make open_purge_table(), close_thread_tables() accessible from InnoDB
purge subsystem

trx_purge_table_open(): Modified to open TABLE* using the
coordinator's MDL ticket

open_purge_table(): Accept and use the MDL ticket from coordinator thread

purge_sys_t::coordinator_thd: To track the coordinator thread in purge
subsystem

purge_node_t::end(): Skip innobase_reset_background_thd() for the coordinator
thread to prevent the premature table closure

trx_purge(): Closes the coordinator's TABLE* objects after all workers
are completed
ba774a0a90fa...
ba774a0a90fac0163babe9d7a964aa36503e1711
Sergei Petrunia
Fix in condition pushdown code: don't pass List<Field_pair> by value.

Affected functions:
  find_matching_field_pair(Item *item, List<Field_pair> pair_list)
  get_corresponding_field_pair(Item *item, List<Field_pair> pair_list)

Both only traverse the pair_list.
They use List_iterator so we can't easily switch to using const-reference.
b9f0e7a6ccba...
b9f0e7a6ccba20bd2fa76e88f7812a6252c3372d
Yuchen Pei
MDEV-38752 [wip] check supertype

- Type_handler_general_purpose_int
- Type_handler_decimal_result
- Type_handler_longstr
b8672c7b81fd...
b8672c7b81fda01476657e646e30e99237b5aa6d
Dave Gosselin
MDEV-39209: use iterative cleanup for merged units to avoid stack overflow

Query optimization can merge derived tables (VIEWs being a type of derived
table) into outer queries, leaving behind stranded  st_select_lex_unit objects
("stranded units") for post-query cleanup.

Previously, these were cleaned up recursively. For queries with many merged
derived tables, the deep recursion over the list of stranded units could
exhaust the stack. This change replaces the recursive cleanup with an
iterative loop to prevent stack overflows.
a7171cb9484c...
a7171cb9484c9cefe75a8bc92bd13ecf318983ba
Michal Schorm
CONC-816: remove dead cmake version checks

plugins.cmake had a VERSION_LESS 2.8.11 branch falling back
to include_directories() instead of target_include_directories().
libmariadb/CMakeLists.txt had a VERSION_GREATER 2.8.7 guard
around OBJECT library creation. Both are always true/false
with cmake_minimum_required 3.12.

Co-Authored-By: Claude AI <[email protected]>
925b771fcb74...
925b771fcb7496b0167b937c7806d72ea29274c7
Michal Schorm
CONC-816: remove MSVC_VERSION > 1310 guards in WindowsCache

MSVC_VERSION 1310 is Visual Studio .NET 2003, released in 2003.
The check is always true: cmake 3.12 (our minimum) was released
in 2018 and does not support any MSVC older than VS 2015
(MSVC_VERSION 1900). The guarded functions (strnlen, vsnprintf,
strtok_s) have been available in MSVC since VS 2005 (1400).

Co-Authored-By: Claude AI <[email protected]>
92427f560c18...
92427f560c180595d93962f8bcf76b07ecb8854c
Michal Schorm
CONC-816: ADD_DEFINITIONS -> add_compile_definitions

Replace the deprecated ADD_DEFINITIONS() with the modern
add_compile_definitions() (available since cmake 3.12).
The new form does not require the -D prefix on each definition.

One call passed -Wno-deprecated-declarations (a compiler warning
flag, not a preprocessor definition). ADD_DEFINITIONS() happened
to work because it blindly appends to the compile command line,
but add_compile_definitions() would auto-prepend -D, producing
the nonsensical -D-Wno-deprecated-declarations. Corrected to
add_compile_options(), which is the proper command for compiler
flags.

ADD_DEFINITIONS(${LIBMARIADB_PLUGIN_DEFS}) is left unconverted.
The variable is never set within this repository or mariadb-server
-- presumably set by an external build harness, if at all. Since
ADD_DEFINITIONS expects -DFOO format while add_compile_definitions
expects FOO (without -D), converting without knowing the variable's
content would risk double-prefixing (-D-DFOO).

Co-Authored-By: Claude AI <[email protected]>
8b4f72860e8f...
8b4f72860e8f3538dc5a5fd40a4b9a5b5cec6ab2
Thirunarayanan Balathandayuthapani
MDEV-39261 MariaDB crash on startup in presence of indexed virtual columns

Problem:
========
A single InnoDB purge worker thread can process undo logs from different
tables within the same batch. But get_purge_table(), open_purge_table()
incorrectly assumes that a 1:1 relationship between a purge worker thread
and a table within a single batch. Based on this wrong assumtion,
InnoDB attempts to reuse TABLE objects cached in thd->open_tables for
virtual column computation.

1) Purge worker opens Table A and caches the TABLE pointer in thd->open_tables.
2) Same purge worker moves to Table B in the same batch, get_purge_table()
retrieves the cached pointer for Table A instead of opening Table B.
3) Because innobase::open() is ignored for Table B, the virtual column
template is never initialized.
4) virtual column computation for Table B aborts the server

Solution:
========
The purge coordinator thread now opens both InnoDB dict_table_t* and
MariaDB TABLE* handles during batch preparation in
trx_purge_attach_undo_recs(). These handles are stored in a new
purge_table_ctx_t structure within each purge_node_t->tables map,
keyed by table_id. When worker thread needs TABLE* for virtual column
computation, it fetches the TABLE* from purge_node_t->tables.

purge_table_ctx_t(): Structure to hold dict_table_t*, MDL_ticket* and
TABLE* for each table during batch processing. TABLE* is being opened
only when virtual index exists for the table.

innobase_open_purge_table(), innobase_close_thread_table(): wrapper to
make open_purge_table(), close_thread_tables() accessible from InnoDB
purge subsystem

trx_purge_table_open(): Modified to open TABLE* using the
coordinator's MDL ticket

open_purge_table(): Accept and use the MDL ticket from coordinator thread

purge_sys_t::coordinator_thd: To track the coordinator thread in purge
subsystem

purge_node_t::end(): Skip innobase_reset_background_thd() for the coordinator
thread to prevent the premature table closure

trx_purge(): Closes the coordinator's TABLE* objects after all workers
are completed
7b5d26c1f1c0...
7b5d26c1f1c0b2f1e9a1c324cf9e963b4f7b8a99
Georg Richter
Merge branch '3.3' into 3.4
  • cc-x-codbc-windows: 'dojob pwd if '3.4' == '3.4' ls win32/test SET TEST_DSN=master SET TEST_DRIVER=master SET TEST_PORT=3306 SET TEST_SCHEMA=odbcmaster if '3.4' == '3.4' cd win32/test if '3.4' == '3.4' ctest --output-on-failure' failed -  stdio
7aa57104989a...
7aa57104989aa78bb9e80c1e1defc467600da4b4
Michal Schorm
CONC-816: CMAKE_COMPILER_IS_GNUCC -> CMAKE_C_COMPILER_ID

CMAKE_COMPILER_IS_GNUCC is deprecated since cmake 2.6.
The same file already uses CMAKE_C_COMPILER_ID at line 414;
this makes the GCC detection consistent.

Co-Authored-By: Claude AI <[email protected]>
7a7799e68a3a...
7a7799e68a3a9525a9750efaa4a289eed6ed4326
forkfun
MDEV-30295 mysqldump produces syntactically incorrect statement

Remove version-specific executable comments, that were used for backward compatibility
with old MySQL versions (3.2, 4.0, 4.1, 5.0).
6be23fd1fb75...
6be23fd1fb75e79c0f87b1ef7adb3d7c4257b296
Thirunarayanan Balathandayuthapani
MDEV-39261 MariaDB crash on startup in presence of indexed virtual columns

Problem:
========
A single InnoDB purge worker thread can process undo logs from different
tables within the same batch. But get_purge_table(), open_purge_table()
incorrectly assumes that a 1:1 relationship between a purge worker thread
and a table within a single batch. Based on this wrong assumtion,
InnoDB attempts to reuse TABLE objects cached in thd->open_tables for
virtual column computation.

1) Purge worker opens Table A and caches the TABLE pointer in thd->open_tables.
2) Same purge worker moves to Table B in the same batch, get_purge_table()
retrieves the cached pointer for Table A instead of opening Table B.
3) Because innobase::open() is ignored for Table B, the virtual column
template is never initialized.
4) virtual column computation for Table B aborts the server

Solution:
========
The purge coordinator thread now opens both InnoDB dict_table_t* and
MariaDB TABLE* handles during batch preparation in
trx_purge_attach_undo_recs(). These handles are stored in a new
purge_table_ctx_t structure within each purge_node_t->tables map,
keyed by table_id. When worker thread needs TABLE* for virtual column
computation, it fetches the TABLE* from purge_node_t->tables.

purge_table_ctx_t(): Structure to hold dict_table_t*, MDL_ticket* and
TABLE* for each table during batch processing. TABLE* is being opened
only when virtual index exists for the table.

innobase_open_purge_table(), innobase_close_thread_table(): wrapper to
make open_purge_table(), close_thread_tables() accessible from InnoDB
purge subsystem

trx_purge_table_open(): Modified to open TABLE* using the
coordinator's MDL ticket

open_purge_table(): Accept and use the MDL ticket from coordinator thread

purge_sys_t::coordinator_thd: To track the coordinator thread in purge
subsystem

purge_node_t::end(): Skip innobase_reset_background_thd() for the coordinator
thread to prevent the premature table closure

trx_purge(): Closes the coordinator's TABLE* objects after all workers
are completed
698b68f1fed9...
698b68f1fed90b30bdbf755454f15f7d95816a53
Marko Mäkelä
Merge MDEV-21423
679842888710...
67984288871061a3ac57c62a94985868ea2555f6
Georg Richter
Fix MARIADB_TIMESTAMP calculation and packing (CONC-815)

- Add a new type MARIADB_TIMESTAMP which includes seconds and
  microseconds since the epoch.
- Fix second and microsecond calculation (seconds are now
  packed in MyISAM format).
- Update documentation to reflect the new type
- Updated Replication/Binlog section of client/server protocol
  documentation
- Add rpl_api tests for automated testing (requires binary
  log enabled on the server).

Credits to Asim Viladi Oglu Manizada who discovered this bug!
  • cc-x-codbc-windows: 'dojob pwd if '3.4' == '3.4' ls win32/test SET TEST_DSN=master SET TEST_DRIVER=master SET TEST_PORT=3306 SET TEST_SCHEMA=odbcmaster if '3.4' == '3.4' cd win32/test if '3.4' == '3.4' ctest --output-on-failure' failed -  stdio
665467ea2e2b...
665467ea2e2ba30f6dfca6f1916429e5f43c5d76
Alexey Botchkov
MDEV-39124 XMLTYPE: allow only well-formed XML.

Necessary checks added to the XMLTYPE.
64ec6c243cae...
64ec6c243cae2305f7906b5688fbebaa0ac6b3df
Aleksey Midenkov
Vanilla: TimestampString for printing timestamps
64af723c406f...
64af723c406f6d53edcbdf7f63bd42464898841a
forkfun
MDEV-30295 mysqldump produces syntactically incorrect statement

Remove version-specific executable comments, that were used for backward compatibility
with old MySQL versions (3.2, 4.0, 4.1, 5.0).
60d16688d178...
60d16688d178e5e9f4f3514976a7b0cc4132ac33
Thirunarayanan Balathandayuthapani
MDEV-39261 MariaDB crash on startup in presence of indexed virtual columns

Problem:
========
A single InnoDB purge worker thread can process undo logs from different
tables within the same batch. But get_purge_table(), open_purge_table()
incorrectly assumes that a 1:1 relationship between a purge worker thread
and a table within a single batch. Based on this wrong assumtion,
InnoDB attempts to reuse TABLE objects cached in thd->open_tables for
virtual column computation.

1) Purge worker opens Table A and caches the TABLE pointer in thd->open_tables.
2) Same purge worker moves to Table B in the same batch, get_purge_table()
retrieves the cached pointer for Table A instead of opening Table B.
3) Because innobase::open() is ignored for Table B, the virtual column
template is never initialized.
4) virtual column computation for Table B aborts the server

Solution:
========
The purge coordinator thread now opens both InnoDB dict_table_t* and
MariaDB TABLE* handles during batch preparation in
trx_purge_attach_undo_recs(). These handles are stored in a new
purge_table_ctx_t structure within each purge_node_t->tables map,
keyed by table_id. When worker thread needs TABLE* for virtual column
computation, it fetches the TABLE* from purge_node_t->tables.

purge_table_ctx_t(): Structure to hold dict_table_t*, MDL_ticket* and
TABLE* for each table during batch processing. TABLE* is being opened
only when virtual index exists for the table.

innobase_open_purge_table(), innobase_close_thread_table(): wrapper to
make open_purge_table(), close_thread_tables() accessible from InnoDB
purge subsystem

trx_purge_table_open(): Modified to open TABLE* using the
coordinator's MDL ticket

open_purge_table(): Accept and use the MDL ticket from coordinator thread

purge_sys_t::coordinator_thd: To track the coordinator thread in purge
subsystem

purge_node_t::end(): Skip innobase_reset_background_thd() for the coordinator
thread to prevent the premature table closure

trx_purge(): Closes the coordinator's TABLE* objects after all workers
are completed
5658137eabe8...
5658137eabe87614dabcf3b3c3b468ef7b6c1772
Georg Richter
Merge pull request #305 from FaramosCZ/CONC-813+CONC-814

CONC-813, CONC-814 Fix UBSan errors
3c4456927d26...
3c4456927d2696c6a1ff29959e969d78ffe56083
Michal Schorm
CONC-816: drop obsolete cmake policy loop

All policies except CMP0077 already default to NEW with
cmake_minimum_required(VERSION 3.12). Keep only the explicit
CMP0077 set until the minimum is bumped to 3.13.

Co-Authored-By: Claude AI <[email protected]>
2d3c173ad54f...
2d3c173ad54f6fdb100dc7df5859b1d6c5da99cc
Yuchen Pei
MDEV-38752 [wip] check supertype

- Type_handler_general_purpose_int
- Type_handler_decimal_result
- Type_handler_longstr
- time_common
- datetime_common
- date_common
227427a04bba...
227427a04bba9647144241e23561a5eb56ef095f
Thirunarayanan Balathandayuthapani
MDEV-39261 MariaDB crash on startup in presence of indexed virtual columns

Problem:
========
A single InnoDB purge worker thread can process undo logs from different
tables within the same batch. But get_purge_table(), open_purge_table()
incorrectly assumes that a 1:1 relationship between a purge worker thread
and a table within a single batch. Based on this wrong assumtion,
InnoDB attempts to reuse TABLE objects cached in thd->open_tables for
virtual column computation.

1) Purge worker opens Table A and caches the TABLE pointer in thd->open_tables.
2) Same purge worker moves to Table B in the same batch, get_purge_table()
retrieves the cached pointer for Table A instead of opening Table B.
3) Because innobase::open() is ignored for Table B, the virtual column
template is never initialized.
4) virtual column computation for Table B aborts the server

Solution:
========
The purge coordinator thread now opens both InnoDB dict_table_t* and
MariaDB TABLE* handles during batch preparation in
trx_purge_attach_undo_recs(). These handles are stored in a new
purge_table_ctx_t structure within each purge_node_t->tables map,
keyed by table_id. When worker thread needs TABLE* for virtual column
computation, it fetches the TABLE* from purge_node_t->tables.

purge_table_ctx_t(): Structure to hold dict_table_t*, MDL_ticket* and
TABLE* for each table during batch processing. TABLE* is being opened
only when virtual index exists for the table.

innobase_open_purge_table(), innobase_close_thread_table(): wrapper to
make open_purge_table(), close_thread_tables() accessible from InnoDB
purge subsystem

trx_purge_table_open(): Modified to open TABLE* using the
coordinator's MDL ticket

open_purge_table(): Accept and use the MDL ticket from coordinator thread

purge_sys_t::coordinator_thd: To track the coordinator thread in purge
subsystem

purge_node_t::end(): Skip innobase_reset_background_thd() for the coordinator
thread to prevent the premature table closure

trx_purge(): Closes the coordinator's TABLE* objects after all workers
are completed
1ac35877e847...
1ac35877e8479a0ea7a91c76b48fcb1bbe1c02f9
Aleksey Midenkov
Vanilla: converted COMBINE macro to interval2usec inline function
10d2dfa2101a...
10d2dfa2101a2609007f29f4f837fafc3a221906
Abhishek Bansal
MDEV-38474: ASAN heap-use-after-free in st_select_lex_unit::cleanup

cleanup_stranded_units() was added at the start of
st_select_lex_unit::cleanup() by 34a8209d6657. This causes a
use-after-free when nested subqueries are merged into their parent
unit. With nested subqueries like:

  SELECT * FROM t1
  WHERE a IN (SELECT b FROM t2
              WHERE a IN (SELECT c FROM t3 WHERE FALSE HAVING c < 0));

the stranded_clean_list chains the units as: Unit1 -> Unit2 -> Unit3.
Because cleanup_stranded_units() was called first, Unit1->cleanup()
would recursively trigger Unit2->cleanup(), which in turn would
trigger Unit3->cleanup(). Unit3's cleanup frees its heap-allocated
join structures. But since Unit3 was merged into Unit2, Unit2 still
holds references to Unit3's structures (e.g., st_join_table). When
control returns to Unit2 for its own local cleanup, it accesses
already-freed memory.

Fix: move cleanup_stranded_units() to the end of cleanup(). This way,
each unit completes its own local cleanup first—clearing its
references to any child structures—before triggering cleanup of its
stranded (child) units. This enforces a parent-first cleanup order.
BuildBot (0.8.9) working for the MariaDB project.
Page built: Fri 10 Apr 2026 02:41:03 (UTC)