Valgrind error on main.audit_log_threadpool

Bug #1650322 reported by Laurynas Biveinis
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Percona Server moved to
Status tracked in 5.7
Fix Released
Sergei Glushchenko

Bug Description

On 5.6 trunk, Valgrind build:

$ ./mtr --debug-server --valgrind-mysqld audit_log_threadpool --valgrind-option=--track-origins=yes

==7496== Thread 21:
==7496== Conditional jump or move depends on uninitialised value(s)
==7496== at 0x77D715: mysql_rewrite_query(THD*) (
==7496== by 0x664506: mysql_audit_general_log(THD*, char const*, unsigned int, char const*, unsigned long) (sql_audit.h:91)
==7496== by 0x66B07F: LOGGER::log_command(THD*, enum_server_command, char const*, unsigned long) (
==7496== by 0x66B193: general_log_print(THD*, enum_server_command, char const*, ...) (
==7496== by 0x6C2447: acl_authenticate(THD*, unsigned int) (
==7496== by 0x7069FA: check_connection(THD*) (
==7496== by 0x706BC4: login_connection(THD*) (
==7496== by 0x83A68C: threadpool_add_connection(THD*) (
==7496== by 0x83DDB7: handle_event(connection_t*) (
==7496== by 0x83E075: worker_main(void*) (
==7496== by 0xE5A984: pfs_spawn_thread (
==7496== by 0x58A06C9: start_thread (pthread_create.c:333)
==7496== by 0x64670AE: clone (clone.S:105)
==7496== Uninitialised value was created by a heap allocation
==7496== at 0x4C2CFCF: operator new(unsigned long) (vg_replace_malloc.c:332)
==7496== by 0x573CEC: handle_connections_sockets() (
==7496== by 0x572844: mysqld_main(int, char**) (
==7496== by 0x56592F: main (

Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

Not sure if the culprit is audit, threadpool, or something else. Sergei, can you take a look?

tags: added: ci valgrind
Revision history for this message
Sergei Glushchenko (sergei.glushchenko) wrote :

It is thd->lex->contains_plaintext_password which is uninitialized. Usually it is set in lex_start. In this case we probably have not get to sql_parse yet. Lets see what is the difference with the thread-per-connection case.

Revision history for this message
Sergei Glushchenko (sergei.glushchenko) wrote :
Revision history for this message
Shahriyar Rzayev (rzayev-sehriyar) wrote :

Percona now uses JIRA for bug reports so this bug report is migrated to:

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers