Skip to content

MSAN: unwind (fixing clang 22) odbc + connect + sqlite.#942

Merged
RazvanLiviuVarzaru merged 5 commits intoMariaDB:devfrom
grooverdan:msan_odbc_connect_22
Apr 3, 2026
Merged

MSAN: unwind (fixing clang 22) odbc + connect + sqlite.#942
RazvanLiviuVarzaru merged 5 commits intoMariaDB:devfrom
grooverdan:msan_odbc_connect_22

Conversation

@grooverdan
Copy link
Copy Markdown
Member

@grooverdan grooverdan commented Apr 3, 2026

clang Unwind fix unblocks #912

libodbcinst is instrumented for MDBF-1196 and installed.

connect.odbc_sqlite3 triggers:

==34225==WARNING: MemorySanitizer: use-of-uninitialized-value
   #0 0x7f97a01ae354 in foreach_dirinpath /msan-build/libltdl/ltdl.c:702:18
   #1 0x7f97a01ad4ca in find_handle /msan-build/libltdl/ltdl.c:805:8
   #2 0x7f97a01ad4ca in try_dlopen /msan-build/libltdl/ltdl.c:1468:10
   #3 0x7f97a01ac760 in lt_dlopenadvise /msan-build/libltdl/ltdl.c:1667:11
   #4 0x7f97a01ac760 in lt_dlopen /msan-build/libltdl/ltdl.c:1622:10
   #5 0x7f97a01310af in odbc_dlopen /msan-build/DriverManager/SQLConnect.c:842:16
   #6 0x7f97a012e355 in __connect_part_one /msan-build/DriverManager/SQLConnect.c:1182:38
   #7 0x7f97a013dff7 in SQLDriverConnect /msan-build/DriverManager/SQLDriverConnect.c:1403:11
   #8 0x7f97a07b92e1 in ODBConn::DriverConnect(unsigned int) /source/storage/connect/odbconn.cpp:1306:8
   #9 0x7f97a07b057d in ODBConn::Open(char const*, odbc_parms*, unsigned int) /source/storage/connect/odbconn.cpp:1160:11
   #10 0x7f97a07afdb7 in ODBCColumns(_global*, char const*, char const*, char const*, char const*, int, bool, odbc_parms*) /source/storage/connect/odbconn.cpp:353:14
   #11 0x7f97a058ca1e in connect_assisted_discovery(handlerton*, THD*, TABLE_SHARE*, HA_CREATE_INFO*) /source/storage/connect/ha_connect.cc:6037:14
   #12 0x5636f3ef7896 in create_table_impl(THD*, st_ddl_log_state*, st_ddl_log_state*, st_mysql_const_lex_string const&, st_mysql_const_lex_string const&, st_mysql_const_lex_string const&, st_mysql_const_lex_string const&, st_mysql_const_lex_string const&, DDL_options_st, HA_CREATE_INFO*, Alter_info*, int, bool*, st_key**, unsigned int*, st_mysql_const_unsigned_lex_string*) /source/sql/sql_table.cc:4802:13
   #13 0x5636f3ef6048 in mysql_create_table_no_lock(THD*, st_ddl_log_state*, st_ddl_log_state*, Table_specification_st*, Alter_info*, bool*, int, TABLE_LIST*) /source/sql/sql_table.cc:4955:8
   #14 0x5636f3f370ae in mysql_create_table(THD*, TABLE_LIST*, Table_specification_st*, Alter_info*) /source/sql/sql_table.cc:5202:7
   #15 0x5636f3f33879 in Sql_cmd_create_table_like::execute(THD*) /source/sql/sql_table.cc:13222:12
   #16 0x5636f3c77935 in mysql_execute_command(THD*, bool) /source/sql/sql_parse.cc:6201:26
   #17 0x5636f3c684e0 in mysql_parse(THD*, char*, unsigned int, Parser_state*) /source/sql/sql_parse.cc:8223:18
   #18 0x5636f3c618e4 in dispatch_command(enum_server_command, THD*, char*, unsigned int, bool) /source/sql/sql_parse.cc:1924:7
   #19 0x5636f3c6950b in do_command(THD*, bool) /source/sql/sql_parse.cc:1434:17
   #20 0x5636f4090b8c in do_handle_one_connection(CONNECT*, bool) /source/sql/sql_connect.cc:1475:11
   #21 0x5636f4090645 in handle_one_connection /source/sql/sql_connect.cc:1387:5
   #22 0x5636f4be0ce9 in pfs_spawn_thread /source/storage/perfschema/pfs.cc:2201:3
   #23 0x7f97a77edb7a  (/lib/x86_64-linux-gnu/libc.so.6+0x92b7a) (BuildId: 58749c528985eab03e6700ebc1469fa50aa41219)
   #24 0x7f97a786b7f7  (/lib/x86_64-linux-gnu/libc.so.6+0x1107f7) (BuildId: 58749c528985eab03e6700ebc1469fa50aa41219)

But as /etc/odbc* isn't copied to /usr/local/etc this test is skipped on MSAN builders stil.

Copy link
Copy Markdown
Collaborator

@RazvanLiviuVarzaru RazvanLiviuVarzaru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@RazvanLiviuVarzaru RazvanLiviuVarzaru merged commit e90e567 into MariaDB:dev Apr 3, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants