Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-4201 LevelDB Storage Engine MS2
  3. MDEV-4324

LevelDB: Valgrind "Use of uninitialised value" warnings on inserting value into varchar field

    Details

    • Type: Technical task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      ==22818== Thread 22:
      ==22818== Use of uninitialised value of size 8
      ==22818==    at 0x506F0B2: leveldb::crc32c::Extend(unsigned int, char const*, unsigned long) (in leveldb-1.7.0/libleveldb.so.1.7)
      ==22818==    by 0x50570C3: leveldb::log::Writer::EmitPhysicalRecord(leveldb::log::RecordType, char const*, unsigned long) (in leveldb-1.7.0/libleveldb.so.1.7)
      ==22818==    by 0x50572B3: leveldb::log::Writer::AddRecord(leveldb::Slice const&) (in leveldb-1.7.0/libleveldb.so.1.7)
      ==22818==    by 0x5051879: leveldb::DBImpl::Write(leveldb::WriteOptions const&, leveldb::WriteBatch*) (in leveldb-1.7.0/libleveldb.so.1.7)
      ==22818==    by 0xD4DE92: Ldb_transaction::flush_batch_intern() (ha_leveldb.cc:265)
      ==22818==    by 0xD4DF20: Ldb_transaction::flush_batch() (ha_leveldb.cc:274)
      ==22818==    by 0xD4DDFF: Ldb_transaction::commit() (ha_leveldb.cc:251)
      ==22818==    by 0xD4CB9F: ha_leveldb::external_lock(THD*, int) (ha_leveldb.cc:2179)
      ==22818==    by 0x623716: handler::ha_external_lock(THD*, int) (handler.cc:7067)
      ==22818==    by 0x8FDB5F: unlock_external(THD*, TABLE**, unsigned int) (lock.cc:646)
      ==22818==    by 0x8FD1C0: mysql_unlock_tables(THD*, st_mysql_lock*) (lock.cc:389)
      ==22818==    by 0x738B90: close_thread_tables(THD*) (sql_base.cc:1456)
      ==22818== 
      ==22818== Use of uninitialised value of size 8
      ==22818==    at 0x506F1EF: leveldb::crc32c::Extend(unsigned int, char const*, unsigned long) (in leveldb-1.7.0/libleveldb.so.1.7)
      ==22818==    by 0x50570C3: leveldb::log::Writer::EmitPhysicalRecord(leveldb::log::RecordType, char const*, unsigned long) (in leveldb-1.7.0/libleveldb.so.1.7)
      ==22818==    by 0x50572B3: leveldb::log::Writer::AddRecord(leveldb::Slice const&) (in leveldb-1.7.0/libleveldb.so.1.7)
      ==22818==    by 0x5051879: leveldb::DBImpl::Write(leveldb::WriteOptions const&, leveldb::WriteBatch*) (in leveldb-1.7.0/libleveldb.so.1.7)
      ==22818==    by 0xD4DE92: Ldb_transaction::flush_batch_intern() (ha_leveldb.cc:265)
      ==22818==    by 0xD4DF20: Ldb_transaction::flush_batch() (ha_leveldb.cc:274)
      ==22818==    by 0xD4DDFF: Ldb_transaction::commit() (ha_leveldb.cc:251)
      ==22818==    by 0xD4CB9F: ha_leveldb::external_lock(THD*, int) (ha_leveldb.cc:2179)
      ==22818==    by 0x623716: handler::ha_external_lock(THD*, int) (handler.cc:7067)
      ==22818==    by 0x8FDB5F: unlock_external(THD*, TABLE**, unsigned int) (lock.cc:646)
      ==22818==    by 0x8FD1C0: mysql_unlock_tables(THD*, st_mysql_lock*) (lock.cc:389)
      ==22818==    by 0x738B90: close_thread_tables(THD*) (sql_base.cc:1456)
      ==22818== 
      
      revision-id: psergey@askmonty.org-20130322111245-43wqvaouclxso5zo
      revno: 4807
      branch-nick: mysql-5.6-leveldb
      

      Test case:

      CREATE TABLE t1 (pk INT PRIMARY KEY, c VARCHAR(4)) ENGINE=LevelDB;
      INSERT INTO t1 VALUES (1,'foo'), (2,'bar'); 
      

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            psergey Sergei Petrunia added a comment -

            Not repeatable after latest valgrind fixes. Added a testcase.

            Show
            psergey Sergei Petrunia added a comment - Not repeatable after latest valgrind fixes. Added a testcase.

              People

              • Assignee:
                psergey Sergei Petrunia
                Reporter:
                elenst Elena Stepanova
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 15 minutes
                  15m