1. 27 May, 2010 3 commits
  2. 26 May, 2010 5 commits
    • Tor Didriksen's avatar
      Bug #53445 Build with -Wall and fix warnings that it generates · 85da8956
      Tor Didriksen authored
      Add -Wall to gcc/g++
      Fix most warnings reported in dbg and opt mode.
      
      
      cmd-line-utils/libedit/filecomplete.c:
        Remove unused auto variables.
      configure.cmake:
        Add -Wall to gcc.
      extra/comp_err.c:
        Cast to correct type.
      extra/perror.c:
        Fix segfault (but warnings about deprecated features remain)
      extra/yassl/taocrypt/include/runtime.hpp:
        Comparing two literals was reported as undefined behaviour.
      include/my_global.h:
        Add a template for aligning character buffers.
      mysys/lf_alloc-pin.c:
        Initialize pointer.
      sql/mysqld.cc:
        Use UNINIT_VAR rather than LINT_INIT.
      sql/partition_info.cc:
        Use UNINIT_VAR rather than LINT_INIT.
      sql/rpl_handler.cc:
        Use char[] rather than unsigned long[] array for placement buffer.
      sql/spatial.cc:
        Use char[] rather than unsigned void*[] array for placement buffer.
      sql/spatial.h:
        Use char[] rather than unsigned void*[] array for placement buffer.
      sql/sql_partition.cc:
        Initialize auto variable.
      sql/sql_table.cc:
        Initialize auto variables.
        Add parens around assignment within if()
      sql/sys_vars.cc:
        Use UNINIT_VAR.
      storage/innobase/os/os0file.c:
        Init first slot in auto variable.
      storage/myisam/mi_create.c:
        Use UNINIT_VAR rather than LINT_INIT.
      storage/myisam/mi_open.c:
        Remove (wrong) casting.
      storage/myisam/mi_page.c:
        Remove (wrong) casting.
      storage/myisam/mi_search.c:
        Cast to uchar* rather than char*.
      strings/ctype-ucs2.c:
        Use UNINIT_VAR rather than LINT_INIT.
        Add (uchar*) casting.
      85da8956
    • Alexander Nozdrin's avatar
    • Alexander Nozdrin's avatar
      de0f63c1
    • Mattias Jonsson's avatar
      pre push fix of test results · 3fb49262
      Mattias Jonsson authored
      3fb49262
    • Jimmy Yang's avatar
      Check in the support for Information Schema System Table Views. Users · 60d42882
      Jimmy Yang authored
      can now view the content of InnoDB System Tables through following
      information schema tables:
      
      information_schema.INNODB_SYS_TABLES
      information_schema.INNODB_SYS_INDEXES
      information_schema.INNODB_SYS_COUMNS
      information_schema.INNODB_SYS_FIELDS
      information_schema.INNODB_SYS_FOREIGN
      information_schema.INNODB_SYS_FOREIGN_COLS
      information_schema.INNODB_SYS_TABLESTATS
      
      rb://330 Approved by Marko
      60d42882
  3. 25 May, 2010 7 commits
  4. 24 May, 2010 6 commits
    • Mattias Jonsson's avatar
      merge · 8b18b6d7
      Mattias Jonsson authored
      8b18b6d7
    • Vasil Dimov's avatar
      Merge a changeset from mysql-5.1-innodb: · c4845781
      Vasil Dimov authored
        ------------------------------------------------------------
        revno: 3477
        revision-id: vasil.dimov@oracle.com-20100520132735-a120y85kqmhp7hxi
        parent: vasil.dimov@oracle.com-20100520125403-3u9ydcfu8vz8spls
        committer: Vasil Dimov <vasil.dimov@oracle.com>
        branch nick: mysql-5.1-innodb
        timestamp: Thu 2010-05-20 16:27:35 +0300
        message:
          Disable main.ps_3innodb for valgrind tests since it results in known
          failures, that are described in
          
          Bug#38999 valgrind warnings for update statement in function compare_record()
          
          At the time I am adding this the failures are:
          
          main.ps_3innodb                          [ fail ]  Found warnings/errors in server log file!
                  Test ended at 2010-05-20 01:17:34
          line
          ==31559== Thread 11:
          ==31559== Conditional jump or move depends on uninitialised value(s)
          ==31559==    at 0x75C5BD: compare_record(st_table*) (sql_update.cc:35)
          ==31559==    by 0x744732: write_record(THD*, st_table*, st_copy_info*) (sql_insert.cc:1486)
          ==31559==    by 0x74A0D7: mysql_insert(THD*, TABLE_LIST*, List<Item>&, List<List<Item> >&, List<Item>&, List<Item>&, enum_duplicates, bool) (sql_insert.cc:835)
          ==31559==    by 0x6A79B4: mysql_execute_command(THD*) (sql_parse.cc:3198)
          ==31559==    by 0x754998: Prepared_statement::execute(String*, bool) (sql_prepare.cc:3583)
          ==31559==    by 0x754C4F: Prepared_statement::execute_loop(String*, bool, unsigned char*, unsigned char*) (sql_prepare.cc:3258)
          ==31559==    by 0x754F33: mysql_sql_stmt_execute(THD*) (sql_prepare.cc:2529)
          ==31559==    by 0x6A5028: mysql_execute_command(THD*) (sql_parse.cc:2272)
          ==31559==    by 0x6ADAE8: mysql_parse(THD*, char const*, unsigned, char const**) (sql_parse.cc:5986)
          ==31559==    by 0x6AF3A4: dispatch_command(enum_server_command, THD*, char*, unsigned) (sql_parse.cc:1233)
          ==31559==    by 0x6B0800: do_command(THD*) (sql_parse.cc:874)
          ==31559==    by 0x69CB46: handle_one_connection (sql_connect.cc:1134)
          ==31559==    by 0x33EDA062F6: start_thread (in /lib64/libpthread-2.5.so)
          ==31559==    by 0x33ECED1B6C: clone (in /lib64/libc-2.5.so)
          ==31559== Conditional jump or move depends on uninitialised value(s)
          ==31559==    at 0x75C5D0: compare_record(st_table*) (sql_update.cc:35)
          ==31559==    by 0x744732: write_record(THD*, st_table*, st_copy_info*) (sql_insert.cc:1486)
          ==31559==    by 0x74A0D7: mysql_insert(THD*, TABLE_LIST*, List<Item>&, List<List<Item> >&, List<Item>&, List<Item>&, enum_duplicates, bool) (sql_insert.cc:835)
          ==31559==    by 0x6A79B4: mysql_execute_command(THD*) (sql_parse.cc:3198)
          ==31559==    by 0x754998: Prepared_statement::execute(String*, bool) (sql_prepare.cc:3583)
          ==31559==    by 0x754C4F: Prepared_statement::execute_loop(String*, bool, unsigned char*, unsigned char*) (sql_prepare.cc:3258)
          ==31559==    by 0x754F33: mysql_sql_stmt_execute(THD*) (sql_prepare.cc:2529)
          ==31559==    by 0x6A5028: mysql_execute_command(THD*) (sql_parse.cc:2272)
          ==31559==    by 0x6ADAE8: mysql_parse(THD*, char const*, unsigned, char const**) (sql_parse.cc:5986)
          ==31559==    by 0x6AF3A4: dispatch_command(enum_server_command, THD*, char*, unsigned) (sql_parse.cc:1233)
          ==31559==    by 0x6B0800: do_command(THD*) (sql_parse.cc:874)
          ==31559==    by 0x69CB46: handle_one_connection (sql_connect.cc:1134)
          ==31559==    by 0x33EDA062F6: start_thread (in /lib64/libpthread-2.5.so)
          ==31559==    by 0x33ECED1B6C: clone (in /lib64/libc-2.5.so)
          ^ Found warnings in /export/home4/pb2/test/sb_3-1827397-1274300957.87/mysql-5.1.48-linux-x86_64-test/mysql-test/var-n_mix/log/mysqld.1.err
      c4845781
    • Vasil Dimov's avatar
      Disable the tests on embedded windows that fail due to · 3990cb93
      Vasil Dimov authored
      Bug #53947 InnoDB: Assertion failure in thread 4224 in file
      .\sync\sync0sync.c line 324
      
      This changeset should be reverted once the bug is fixed.
      3990cb93
    • Vasil Dimov's avatar
    • Mattias Jonsson's avatar
      Merge of bug#51851. · 7af0bbc3
      Mattias Jonsson authored
      Also moved HA_DATA_PARTITION from ha_partition.cc to table.h.
      7af0bbc3
    • Marko Mäkelä's avatar
      Merge a change from mysql-5.1-innodb: · 68ce054f
      Marko Mäkelä authored
      ------------------------------------------------------------
      revno: 3479
      revision-id: marko.makela@oracle.com-20100524110439-fazi70rlmt07tzd9
      parent: vasil.dimov@oracle.com-20100520133157-42uk5q3pp0vsinac
      committer: Marko Mäkelä <marko.makela@oracle.com>
      branch nick: 5.1-innodb
      timestamp: Mon 2010-05-24 14:04:39 +0300
      message:
        Bug#53578: assert on invalid page access, in fil_io()
      
        Store the max_space_id in the data dictionary header in order to avoid
        space_id reuse.
      
        DICT_HDR_MIX_ID: Renamed to DICT_HDR_MAX_SPACE_ID, DICT_HDR_MIX_ID_LOW.
      
        dict_hdr_get_new_id(): Return table_id, index_id, space_id or a subset of them.
      
        fil_system_t: Add ibool space_id_reuse_warned.
      
        fil_create_new_single_table_tablespace(): Get the space_id from the caller.
      
        fil_space_create(): Issue a warning if the fil_system->max_assigned_id
        is exceeded.
      
        fil_assign_new_space_id(): Return TRUE/FALSE and take a pointer to the
        space_id as a parameter. Make the function public.
      
        fil_init(): Initialize all fil_system fields by mem_zalloc(). Remove
        explicit initializations of certain fields to 0 or NULL.
      68ce054f
  5. 23 May, 2010 1 commit
  6. 21 May, 2010 10 commits
    • Calvin Sun's avatar
    • Vasil Dimov's avatar
      Use the correct len instead of sizeof(void) in posix_fadvise() · 6788201e
      Vasil Dimov authored
      Also explain in the comment the units of the "offset" parameter
      6788201e
    • Vasil Dimov's avatar
      Move os_file_write() before posix_fadvise(POSIX_FADV_DONTNEED). · 5862a54b
      Vasil Dimov authored
      It is wrong to tell the OS that a block is not going to be accessed and
      write to it immediately afterwards.
      5862a54b
    • Vasil Dimov's avatar
      Fix comment on row_merge_write() · 34096d47
      Vasil Dimov authored
      34096d47
    • Vasil Dimov's avatar
    • Vasil Dimov's avatar
      5fff85e6
    • Jon Olav Hauglid's avatar
      merge from mysql-trunk-runtime · 923da1e8
      Jon Olav Hauglid authored
      923da1e8
    • Alexander Nozdrin's avatar
      Backporting a test case for Bug#30036 (SHOW TABLE TYPES causes the debug client · 5e4a4020
      Alexander Nozdrin authored
      to crash) from mysql-6.0-codebase.
      5e4a4020
    • Alexander Nozdrin's avatar
      Fix for Bug#52923 (Inadequate documentation of "Can't get hostname for your address" error). · 9003fb7c
      Alexander Nozdrin authored
      The thing is that on some platforms (e.g. Mac OS X) sockaddr_in / sockaddr_in6
      contain a non-standard field (sin_len / sin6_len), that must be set.
      
      The problem was that only standard fields were set, thus getnameinfo() returned
      EAI_SYSTEM instead of EAI_NONAME.
      
      The fix is to introduce configure-time checks (for GNU auto-tools and CMake) for
      those additional fields and to set them if they are available.
      9003fb7c
    • Dmitry Lenev's avatar
      Follow-up for the fix for bug #46947 "Embedded SELECT without · 705b98df
      Dmitry Lenev authored
      FOR UPDATE is causing a lock".
       
      This patch tries to address problems which were exposed 
      during backporting of original patch to 5.1 tree.
       
      - It ensures that we don't change locking behavior of simple
        SELECT statements on InnoDB tables when they are executed
        under LOCK TABLES ... READ and with @@innodb_table_locks=0.
        Also we no longer pass TL_READ_DEFAULT/TL_WRITE_DEFAULT 
        lock types, which are supposed to be parser-only, to 
        handler::start_stmt() method.
      - It makes check_/no_concurrent_insert.inc auxiliary scripts 
        more robust against changes in test cases that use them
        and also ensures that they don't unnecessarily change 
        environment of caller.
      
      mysql-test/include/check_concurrent_insert.inc:
        Reset DEBUG_SYNC facility before and after using it in
        auxiliary script. This makes this script more robust against
        changes in test cases calling it. It also ensures that script
        does not unnecessarily change environment of caller.
      mysql-test/include/check_no_concurrent_insert.inc:
        Reset DEBUG_SYNC facility before and after using it in
        auxiliary script. This makes this script more robust against
        changes in test cases calling it. It also ensures that script
        does not unnecessarily change environment of caller.
      mysql-test/r/innodb-lock.result:
        Added coverage for LOCK TABLES ... READ behavior in
        @@innodb_table_locks = 0 mode. This test also checks
        that an appropriate type of lock is passed to
        handler::start_stmt() method.
      mysql-test/t/innodb-lock.test:
        Added coverage for LOCK TABLES ... READ behavior in
        @@innodb_table_locks = 0 mode. This test also checks
        that an appropriate type of lock is passed to
        handler::start_stmt() method.
      sql/sql_base.cc:
        Since we no longer set TL_READ as lock type for tables used  
        in simple SELECT right in the parser, in order to preserve  
        behavior for such statements on InnoDB tables when in  
        LOCK TABLES mode with @innodb_table_locks = 0,  
        check_lock_and_start_stmt() had to be changed to convert  
        TL_READ_DEFAULT to an appropriate type of read lock before  
        passing it to handler::start_stmt() method.  
        We do similar thing for TL_WRITE_DEFAULT as this lock type  
        is also supposed to be parser-only type.  
        As consequence read_lock_type_for_table() had to be  
        adjusted to behave properly when it is called from  
        check_lock_and_start_stmt() in prelocked mode.
      705b98df
  7. 20 May, 2010 4 commits
    • Marko Mäkelä's avatar
      Merge a change from mysql-5.1-innodb: · cc182a09
      Marko Mäkelä authored
      ------------------------------------------------------------
      revno: 3475
      revision-id: marko.makela@oracle.com-20100520130734-ueow9mn60czp7o4m
      parent: marko.makela@oracle.com-20100520104042-ma2nsscqdvwoph8k
      committer: Marko Mäkelä <marko.makela@oracle.com>
      branch nick: 5.1-innodb
      timestamp: Thu 2010-05-20 16:07:34 +0300
      message:
        buf_LRU_free_block(): Correct an error in the comment.
      cc182a09
    • Marko Mäkelä's avatar
      Merge a change from mysql-5.1-innodb: · 963b1986
      Marko Mäkelä authored
      TO DO: Enable this in CMake-based builds.
      
      ------------------------------------------------------------
      revno: 3474
      revision-id: marko.makela@oracle.com-20100520104042-ma2nsscqdvwoph8k
      parent: marko.makela@oracle.com-20100519081618-h38q02qxuvcowbtk
      committer: Marko Mäkelä <marko.makela@oracle.com>
      branch nick: 5.1-innodb
      timestamp: Thu 2010-05-20 13:40:42 +0300
      message:
        Bug#53593: Add some instrumentation to improve Valgrind sensitivity
      
        BUILD/*: Add valgrind_configs=--with-valgrind.
        BUILD/*: Remove -USAFEMALLOC from valgrind_flags.
      
        configure.in: Add AC_ARG_WITH(valgrind) and HAVE_VALGRIND.
      
        include/my_sys.h: Define a number of MEM_ wrappers for VALGRIND_ functions.
        include/my_sys.h: Make TRASH do MEM_UNDEFINED().
      
        include/m_string.h: Remove unused macro bzero_if_purify(A,B).
      
        _mymalloc(): Declare MEM_UNDEFINED() on the allocated memory.
      
        _myfree(): Declare MEM_NOACCESS() on the freed memory.
      
        storage/innobase/include/univ.i: Enable UNIV_DEBUG_VALGRIND based on
        HAVE_VALGRIND rather than HAVE_purify.
      
        Possible things to do:
         * In my_global.h, remove the defined(HAVE_purify) condition
           from the _WIN32 uint3korr().
         * In my_global.h *int*korr(), use | instead of +
           in order to keep the Valgrind V bits accurate
         * Consider replacing HAVE_purify with HAVE_VALGRIND
         * Use VALGRIND_CREATE_BLOCK, VALGRIND_DISCARD in mem_root and similar places
      963b1986
    • Alexander Nozdrin's avatar
      Empty merge from mysql-trunk. · 44486663
      Alexander Nozdrin authored
      44486663
    • Alexander Nozdrin's avatar
      Auto-merge from mysql-trunk-bugfixing. · a18861e4
      Alexander Nozdrin authored
      a18861e4
  8. 19 May, 2010 4 commits