Bug #28949 n_mix valgrind failing in pushbuild
Submitted: 7 Jun 2007 16:55 Modified: 19 Jun 2007 13:39
Reporter: Mads Martin Joergensen
Status: Closed
Category:Server: Cluster Severity:S1 (Critical)
Version:5.1 OS:Any
Assigned to: Tomas Ulin Target Version:

[7 Jun 2007 16:55] Mads Martin Joergensen
Description:
VALGRIND: 'Invalid read of size 8'
    COUNT: 3
    FUNCTION: get_lock_data(THD*,    FILES:    slave.err
    TESTS:    rpl_ndb_stm_innodb
    STACK: at 0x6168C7: get_lock_data(THD*, st_table**, unsigned, unsigned, st_table**)
(log.h:549)
             by 0x617F4F: mysql_lock_tables(THD*, st_table**, unsigned, unsigned, bool*)
(lock.cc:125)
             by 0x66BF68: lock_tables(THD*, st_table_list*, unsigned, bool*)
(sql_base.cc:4048)
             by 0x74B9D6: ndb_add_ndb_binlog_index(THD*, void*)
(ha_ndbcluster_binlog.cc:2306)
             by 0x754D49: ndb_binlog_thread_func (ha_ndbcluster_binlog.cc:4127)
             by 0x4C2E192: start_thread (in /lib64/libpthread-2.4.so)
             by 0x52AA45C: clone (in /lib64/libc-2.4.so)
           Address 0x54FE388 is 8 bytes inside a block of size 1,472 free'd
             at 0x4A2046E: free (vg_replace_malloc.c:233)
             by 0x6BFDD6: LOGGER::cleanup_base() (log.cc:782)
             by 0x61D2F9: clean_up(bool) (mysqld.cc:1156)
             by 0x621CBC: kill_server(void*) (mysqld.cc:1124)
             by 0x621D1F: kill_server_thread (mysqld.cc:1084)
             by 0x4C2E192: start_thread (in /lib64/libpthread-2.4.so)
             by 0x52AA45C: clone (in /lib64/libc-2.4.so)

How to repeat:
Pushbuild repeats it:

https://intranet.mysql.com/secure/pushbuild/getlog.pl?dir=mysql-5.1-build&entry=jbruehe@tr...
[7 Jun 2007 22:26] Konstantin Osipov
All tight loops must check for thd->killed.
Shutdown thread sets this flag in all THDs.
[16 Jun 2007 11:15] Bugs System
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/28922

ChangeSet@1.2521, 2007-06-16 11:16:02+02:00, tomas@whalegate.ndb.mysql.com +1 -0
  Bug #28949
  - handler binlog's must be shutdoem before logger.cleanup_base
[18 Jun 2007 9:49] Bugs System
Pushed into 5.1.20-beta
[19 Jun 2007 13:39] Jon Stephens
Thank you for your bug report. This issue has been committed to our source repository of
that product and will be incorporated into the next release.

If necessary, you can access the source repository and build the latest available
version, including the bug fix. More information about accessing the source trees is
available at

    http://dev.mysql.com/doc/en/installing-source.html

Documented bugfix in 5.1.20 changelog.
[3 Jul 2007 6:42] Jon Stephens
Also documented for telco-6.2.3 release.