1. 17 Dec, 2007 2 commits
  2. 06 Dec, 2007 1 commit
    • unknown's avatar
      Bug#31752: check strmake() bounds · 08053848
      unknown authored
      post-fixes: prevent semi-related overflow, additional comments
      
      
      mysys/mf_pack.c:
        extra comments
      sql/log.cc:
        prevent overflow (length parameter of strmake() should
        never become < 0)
      sql/sql_show.cc:
        additional comments
      sql/unireg.cc:
        additional comments
      08053848
  3. 26 Nov, 2007 5 commits
    • unknown's avatar
      Bug#31752: check strmake() bounds · 5ce7cdbe
      unknown authored
      strmake() called with wrong parameters:
      5.1-specific fixes.
      
      
      sql/sql_db.cc:
        fix off-by-one buffer-length
      5ce7cdbe
    • unknown's avatar
      Merge mysql.com:/misc/mysql/31752_/50-31752_ · d7d7b10b
      unknown authored
      into  mysql.com:/misc/mysql/31752_/51-31752_
      
      
      client/mysql.cc:
        Auto merged
      sql/log.cc:
        Auto merged
      sql/repl_failsafe.cc:
        Auto merged
      sql/set_var.cc:
        Auto merged
      sql/sql_show.cc:
        Auto merged
      sql/sp.cc:
        manual merge
      strings/strmake.c:
        SCCS merged
      sql/unireg.cc:
        SCCS merged
      d7d7b10b
    • unknown's avatar
      Bug#31752: check strmake() bounds · 1c72446e
      unknown authored
      strmake() called with wrong parameters:
      5.0-specific fixes.
      
      
      client/mysql.cc:
        In debug-mode, strmake() fills unused part of buffer with
        a test-pattern. This overwrites our previous extra '\0'
        (from previous bzero()).
      sql/sp.cc:
        off-by-one buffer-size.
      1c72446e
    • unknown's avatar
      Merge mysql.com:/misc/mysql/31752_/41-31752_ · fe280afa
      unknown authored
      into  mysql.com:/misc/mysql/31752_/50-31752_
      
      
      sql/log.cc:
        Auto merged
      sql/repl_failsafe.cc:
        Auto merged
      sql/set_var.cc:
        Auto merged
      strings/strmake.c:
        Auto merged
      sql/sql_show.cc:
        manually merged
      sql/unireg.cc:
        manual merge
      fe280afa
    • unknown's avatar
      Bug#31752: check strmake() bounds · a905ac34
      unknown authored
      strmake() calls are easy to get wrong. Add checks in extra
      debug mode to identify possible exploits.
      
      Remove some dead code.
      
      Remove some off-by-one errors identified with new checks.
      
      
      sql/log.cc:
        fix off-by-one buffer-length argument to prevent stack smashing
      sql/repl_failsafe.cc:
        fix off-by-one buffer-length argument to prevent stack smashing
      sql/set_var.cc:
        fix off-by-one buffer-length argument to prevent stack smashing
        (already approved, backports #31588)
      sql/sql_show.cc:
        misdimensioned buffers: functions further down the callstack
        expect bufsize of FN_REFLEN
      sql/unireg.cc:
        When EXTRA_DEBUG is enabled, strmake() will write funny patterns to
        buffers it operates on to identify possibly overflows. This leads to
        badness in mysql_create_frm(), so we explicitly put any unused bytes
        (back) into a defined state. Not a bug-fix, but part of the strmake()
        bug detector.
      strings/strmake.c:
        strmake() takes maximum string length rather than buffer-length
        (string length + 1 to accomodate \0 terminator) as argument.
        Since this is easy to get wrong, add extra debug code to identify
        off-by-ones so we can prevent stack smashing.
        
        Alternative "BAD_STRING_COMPILER" removed after checking
        with Monty.
      a905ac34
  4. 23 Nov, 2007 2 commits
    • unknown's avatar
      Merge polly.(none):/home/kaa/src/opt/bug32080/my51-bug26215 · 7ba06230
      unknown authored
      into  polly.(none):/home/kaa/src/opt/mysql-5.1-opt
      
      
      mysql-test/lib/mtr_report.pl:
        Auto merged
      7ba06230
    • unknown's avatar
      Fix for bug #32080: Excessive warnings on Solaris: setrlimit could not · e7501cd9
      unknown authored
      change the size of core files.
      
      Suppress the 'setrlimit could not change the size of the core files'
      warning in mysql-test-run. We do not want core files on some of the
      PushBuild hosts, and PushBuild itself does not set --core-files, so
      that warning is expected.
      
      
      mysql-test/lib/mtr_report.pl:
        Suppress the 'setrlimit could not change the size of the core files'
        warning in mysql-test-run. We do not want core files on some of the
        PushBuild machines, and PushBuild itself does not set --core-files, so
        that warning is expected.
      e7501cd9
  5. 22 Nov, 2007 8 commits
  6. 21 Nov, 2007 4 commits
    • unknown's avatar
      Bug #31171: test alter_table, fulltext2, ps, sp fail, · 0b489d3f
      unknown authored
       "Table is already up to date" vs. "OK"
      
      On MacOSX 10.5 when you cast something to "bool"
      (the built in C type) it takes values 0 or 1 
      instead of 0-255 as it seems to be on older compilers.
      Fixed by removing the typecast (not needed).
      No test case needed : there are tests that test it.
      
      
      storage/myisam/mi_open.c:
        Bug #31171: don't typecast when not needed
      0b489d3f
    • unknown's avatar
      merge of bug 30788 to 5.1-opt · 16f15dba
      unknown authored
      16f15dba
    • unknown's avatar
      Merge magare.gmz:/home/kgeorge/mysql/work/B30788-5.0-opt · 4b910676
      unknown authored
      into  magare.gmz:/home/kgeorge/mysql/work/B30788-5.1-opt
      
      
      mysql-test/r/subselect3.result:
        Auto merged
      sql/item.h:
        Auto merged
      sql/sql_select.cc:
        Auto merged
      mysql-test/r/subselect.result:
        merge of bug 30788 to 5.1-opt
      mysql-test/t/subselect.test:
        merge of bug 30788 to 5.1-opt
      4b910676
    • unknown's avatar
      Bug #30788: Inconsistent retrieval of char/varchar · e9832cee
      unknown authored
      Index lookup does not always guarantee that we can
      simply remove the relevant conditions from the WHERE
      clause. Reasons can be e.g. conversion errors, 
      partial indexes etc. 
      The optimizer was removing these parts of the WHERE 
      condition without any further checking.
      This leads to "false positives" when using indexes.
      Fixed by checking the index reference conditions
      (using WHERE) when using indexes with sub-queries.
      
      
      mysql-test/r/subselect.result:
        Bug #30788: 
         - using where
         - test case
      mysql-test/r/subselect3.result:
        Bug #30788: using where
      mysql-test/t/subselect.test:
        Bug #30788: test case
      sql/item.h:
        Bug #30788: 
         - Declare eq() method of Item_cache descendants : this is used in
         test_if_ref()
         - preserve the field that is being cached for type comparisions
      sql/sql_select.cc:
        Bug #30788: Don't remove the WHERE when using index lookup 
        with subqueries.
      e9832cee
  7. 20 Nov, 2007 6 commits
    • unknown's avatar
      Merge spetrunia@bk-internal.mysql.com:/home/bk/mysql-5.1-opt · 70e22876
      unknown authored
      into  mysql.com:/home/psergey/mysql-5.1-bug30573
      
      
      70e22876
    • unknown's avatar
      merging · 2eaa6352
      unknown authored
      2eaa6352
    • unknown's avatar
      Merge mysql.com:/home/hf/work/32211/my51-32211 · e3c44e3e
      unknown authored
      into  mysql.com:/home/hf/work/31868/my51-31868
      
      
      e3c44e3e
    • unknown's avatar
      Merge mysql.com:/home/hf/work/31960/my51-31960 · abb1afba
      unknown authored
      into  mysql.com:/home/hf/work/31868/my51-31868
      
      
      sql/sql_parse.cc:
        Auto merged
      sql/mysql_priv.h:
        merging
      abb1afba
    • unknown's avatar
      Merge bk@192.168.21.1:mysql-5.1-opt · 3ae44c90
      unknown authored
      into  mysql.com:/home/hf/work/31868/my51-31868
      
      
      BitKeeper/etc/ignore:
        auto-union
      libmysqld/lib_sql.cc:
        Auto merged
      sql/mysqld.cc:
        Auto merged
      sql/sql_table.cc:
        Auto merged
      3ae44c90
    • unknown's avatar
      BUG#30573: Ordered range scan over partitioned tables returns some rows twice · 0b22925c
      unknown authored
      The problem: ha_partition::read_range_first() could return a record that is 
      outside of the scanned range. If that record happened to be in the next 
      subsequent range, it would satisfy the WHERE and appear in the output twice.
      (we would get it the second time when scanning the next subsequent range)
      
      Fix: 
      Made ha_partition::read_range_first() check if the returned recod is within
      the scanned range, like other read_range_first() implementations do.
      
      
      mysql-test/r/partition_range.result:
        BUG#30573: Ordered range scan over partitioned tables returns some rows twice
         - Testcase
      mysql-test/t/partition_range.test:
        BUG#30573: Ordered range scan over partitioned tables returns some rows twice
         - Testcase
      sql/ha_partition.cc:
        BUG#30573: Ordered range scan over partitioned tables returns some rows twice
        - Make ha_partition::read_range_first() check if the returned record is 
          within the range.
      0b22925c
  8. 19 Nov, 2007 7 commits
    • unknown's avatar
      Fix for bug #32221: bug25714, mytest, mysql_client_test complaints and · c4d0901d
      unknown authored
      crashes.
      
      MySQL distributions contain a number of programs that are used only by
      the MySQL test suite internally, i.e. they are not indended to be
      invoked directly by a user. As a result, such programs are not
      documented, do not have any built-in help or proper error reporting,
      which may confuse users.
      
      This patch fixes the problem with the following changes:
      - mytest, libmysqltest and all references to them were removed from the
      distribution since they are not used anymore
      - bug25714 now displays an error message when run with incorrect
      arguments or with the --help option
      - mysql_client_test now does not call abort() in case of errors,
      instead it does a clean exit() with a proper error status.
      
      
      BitKeeper/deleted/.del-mytest.c:
        Delete: libmysql/mytest.c
      BitKeeper/deleted/.del-myTest-package.dsp:
        Delete: VC++Files/libmysqltest/myTest-package.dsp
      BitKeeper/deleted/.del-myTest-package_ia64.dsp:
        Delete: VC++Files/libmysqltest/myTest-package_ia64.dsp
      BitKeeper/deleted/.del-myTest.dsp~4a8c480769193952:
        Delete: VC++Files/libmysqltest/myTest.dsp
      BitKeeper/deleted/.del-myTest.vcproj:
        Delete: VC++Files/libmysqltest/myTest.vcproj
      BitKeeper/deleted/.del-myTest_ia64.dsp:
        Delete: VC++Files/libmysqltest/myTest_ia64.dsp
      BitKeeper/deleted/.del-mytest.c~9a99338689e5de8:
        Delete: VC++Files/libmysqltest/mytest.c
      BitKeeper/deleted/.del-mytest.dsw~2324698861155335:
        Delete: VC++Files/libmysqltest/mytest.dsw
      VC++Files/copy_mysql_files.bat:
        Removed references to libmysqltest.
      libmysql/CMakeLists.txt:
        Removed rules for myTest.
      libmysql/Makefile.am:
        Removed mytest.c from EXTRA_DIST.
      tests/bug25714.c:
        Display a meaningful error message when run incorrectly or with the --help option.
      tests/mysql_client_test.c:
        In case of error, don't cause abnormal program termination, do a clean exit() with an error status.
      c4d0901d
    • unknown's avatar
      Merge polly.(none):/home/kaa/src/opt/mysql-5.0-opt · 83e71330
      unknown authored
      into  polly.(none):/home/kaa/src/opt/mysql-5.1-opt
      
      
      client/mysql.cc:
        Auto merged
      83e71330
    • unknown's avatar
      Merge polly.(none):/home/kaa/src/opt/bug32376/my51-bug26215 · 6a9cc21e
      unknown authored
      into  polly.(none):/home/kaa/src/opt/mysql-5.1-opt
      
      
      client/mysql.cc:
        Auto merged
      6a9cc21e
    • unknown's avatar
      Merge polly.(none):/home/kaa/src/opt/bug32376/my50-bug26215 · ac0612f5
      unknown authored
      into  polly.(none):/home/kaa/src/opt/mysql-5.0-opt
      
      
      client/mysql.cc:
        Auto merged
      ac0612f5
    • unknown's avatar
      Merge polly.(none):/home/kaa/src/opt/bug32376/my50-bug26215 · 43f41d18
      unknown authored
      into  polly.(none):/home/kaa/src/opt/bug32376/my51-bug26215
      
      
      client/mysql.cc:
        Manual merge.
      43f41d18
    • unknown's avatar
      Changed the help text for --comments to make it clear which option · 1f7678fe
      unknown authored
      disables the option explicitely.
      Changed the option location in code so that --help will show it in
      lexical option order.
      
      This is for bug #26215: mysql command line client should not strip
      comments from SQL statements
      
      
      client/mysql.cc:
        Changed the help text for --comments to make it clear which option
        disables the option explicitely.
        Changed the option location in code so that --help will show it in
        lexical option order.
      1f7678fe
    • unknown's avatar
      Bug#32158 Crash in open_table_from_share, on mysql_unpack_partition errors · b1580229
      unknown authored
      added check for the result of mysql_unpack_partition()
      
      
      sql/table.cc:
        added check for the result of mysql_unpack_partition()
      b1580229
  9. 18 Nov, 2007 2 commits
  10. 17 Nov, 2007 3 commits
    • unknown's avatar
      Bug #32211 Test 'windows' and 'windows_shm' failed for embedded server · fe54b274
      unknown authored
      failing 'INSTALL PLUGIN' statement doesn't work in embedded server
      as we disable library loading there.
      Fixed by enabling loading libraries (#define HAVE_DLOPEN), what also
      makes UDF working in the embedded server.
      
      
      include/mysql_embed.h:
        Bug #32211 Test 'windows' and 'windows_shm' failed for embedded server
        
        Let loading libraries in the embedded server
      libmysqld/CMakeLists.txt:
        Bug #32211 Test 'windows' and 'windows_shm' failed for embedded server
        
        let loading libraries in the embedded server
      mysql-test/t/windows.test:
        Bug #32211 Test 'windows' and 'windows_shm' failed for embedded server
        
        make sure proc_1() doesn't exists before we start
      fe54b274
    • unknown's avatar
      Merge gleb.loc:/work/bk/5.0-opt-32335 · f978c7cd
      unknown authored
      into  gleb.loc:/work/bk/5.0-opt
      
      
      f978c7cd
    • unknown's avatar
      Fixed bug #32335. · 99054db6
      unknown authored
      Comparison of a BIGINT NOT NULL column with a constant arithmetic
      expression that evaluates to NULL caused error 1048: "Column '...'
      cannot be null".
      
      Made convert_constant_item() check if the constant expression is NULL
      before attempting to store it in a field. Attempts to store NULL in a
      NOT NULL field caused query errors.
      
      
      sql/item_cmpfunc.cc:
        Fixed bug #32335.
        1. Made convert_constant_item() check if the constant expression is NULL
           before attempting to store it in a field. Attempts to store NULL in
           a NOT NULL field caused query errors.
        
        2. Also minor bug has been fixed: the thd->count_cuted_fields value
           was not restored in case of successful conversion.
      mysql-test/t/select.test:
        Added test case for bug #32335.
      mysql-test/r/select.result:
        Added test case for bug #32335.
      99054db6