Valgrind warning with progress reporting

Bug #882489 reported by Philip Stoev on 2011-10-27
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MariaDB
Incomplete
Medium
Michael Widenius

Bug Description

When executing SHOW PROCESSLIST concurrently with SELECT queries, valgrind reports the following:

==3857== Conditional jump or move depends on uninitialised value(s)
==3857== at 0x6D2916: __printf_fp (in /lib/libc-2.12.1.so)
==3857== by 0x6CD4F9: vfprintf (in /lib/libc-2.12.1.so)
==3857== by 0x6F4C1F: vsnprintf (in /lib/libc-2.12.1.so)
==3857== by 0x6D6C21: snprintf (in /lib/libc-2.12.1.so)
==3857== by 0x82936D3: String::set_real(double, unsigned int, charset_info_st const*) (sql_string.cc:194)
==3857== by 0x828EE44: Protocol_text::store(double, unsigned int, String*) (protocol.cc:1062)
==3857== by 0x842F86D: mysqld_list_processes(THD*, char const*, bool) (sql_show.cc:2022)
==3857== by 0x82AF600: mysql_execute_command(THD*) (sql_parse.cc:3422)
==3857== by 0x82B74E2: mysql_parse(THD*, char*, unsigned int, char const**) (sql_parse.cc:6112)
==3857== by 0x82A98BF: dispatch_command(enum_server_command, THD*, char*, unsigned int) (sql_parse.cc:1221)
==3857== by 0x82A8D1A: do_command(THD*) (sql_parse.cc:916)
==3857== by 0x82A5CFE: handle_one_connection (sql_connect.cc:1191)
==3857== by 0x821918: start_thread (in /lib/libpthread-2.12.1.so)
==3857== by 0x76ACCD: clone (in /lib/libc-2.12.1.so)

line 2022 in sql_show.cc is

      protocol->store(thd_info->progress, 3, &store_buffer);

let me know if a test case or a core file is required.

Changed in maria:
assignee: nobody → Michael Widenius (monty)
milestone: none → 5.3
summary: - Valgrind warning with progress meter
+ Valgrind warning with progress reporting
Changed in maria:
importance: Undecided → High
Sergey Petrunia (sergefp) wrote :

Attempted to repeat by running mysql-test-run --valgrind t/_a.test (will attach the file)
and simultaneously running
while true ; do mysql -uroot -A --socket=./var/tmp/mysqld.1.sock -e 'show processlist'; sleep 2; done

test passed without produce any valgrind warnings.

Please provide a testcase.

Sergey Petrunia (sergefp) wrote :
Changed in maria:
importance: High → Medium
Sergey Petrunia (sergefp) wrote :

Changing importance to Medium as this is something that's not easily repeatable

Changed in maria:
status: New → Incomplete
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers