Bug #17407 valgrind warning in ACC

• Submitted: 15 Feb 2006 0:52
• Modified: 23 May 2006 11:57
• Reporter: Stewart Smith
• Status: Can't repeat
• Category: MySQL Cluster: Cluster (NDB) storage engine
• Severity: S2 (Serious)
• Version: 5.0.19 bk
• OS: 32bit x86 linux
• CPU Architecture: Any

[15 Feb 2006 0:52] Stewart Smith

Description:
==32263==
==32263== Thread 1:
==32263== Conditional jump or move depends on uninitialised value(s)
==32263==    at 0x81EBF5E: Dbtup::handleReadReq(Signal*, Dbtup::Operationrec*, Dbtup::Tablerec*, Dbtup::Page*) (DbtupExecQuery.cpp:873)
==32263==    by 0x81ECA70: Dbtup::execTUPKEYREQ(Signal*) (DbtupExecQuery.cpp:722)
==32263==    by 0x80F342C: SimulatedBlock::executeFunction(unsigned short, Signal*) (SimulatedBlock.hpp:559)
==32263==    by 0x80F35CF: SimulatedBlock::EXECUTE_DIRECT(unsigned, unsigned, Signal*, unsigned) (SimulatedBlock.hpp:722)
==32263==    by 0x81766FB: Dblqh::nextScanConfLoopLab(Signal*) (DblqhMain.cpp:8252)
==32263==    by 0x8176C5F: Dblqh::nextScanConfScanLab(Signal*) (DblqhMain.cpp:8186)
==32263==    by 0x81933B8: Dblqh::execNEXT_SCANCONF(Signal*) (DblqhMain.cpp:6911)
==32263==    by 0x80F342C: SimulatedBlock::executeFunction(unsigned short, Signal*) (SimulatedBlock.hpp:559)
==32263==    by 0x80F35CF: SimulatedBlock::EXECUTE_DIRECT(unsigned, unsigned, Signal*, unsigned) (SimulatedBlock.hpp:722)
==32263==    by 0x80EBD51: Dbacc::sendNextScanConf(Signal*) (DbaccMain.cpp:10295)
==32263==    by 0x80EEC59: Dbacc::checkNextBucketLab(Signal*) (DbaccMain.cpp:9505)
==32263==    by 0x80EF3AA: Dbacc::execACC_CHECK_SCAN(Signal*) (DbaccMain.cpp:9788)

How to repeat:
run ndb_autodiscover with ndbd under valgrind

Suggested fix:
valgrind warnings hurt kittens. we don't want to hurt kittens.

[15 Feb 2006 0:57] Stewart Smith

Further warnings during ndb_basic

==915==
==915== Thread 1:
==915== Conditional jump or move depends on uninitialised value(s)
==915==    at 0x81EBF5E: Dbtup::handleReadReq(Signal*, Dbtup::Operationrec*, Dbtup::Tablerec*, Dbtup::Page*) (DbtupExecQuery.cpp:873)
==915==    by 0x81ECA70: Dbtup::execTUPKEYREQ(Signal*) (DbtupExecQuery.cpp:722)
==915==    by 0x80F342C: SimulatedBlock::executeFunction(unsigned short, Signal*) (SimulatedBlock.hpp:559)
==915==    by 0x80F35CF: SimulatedBlock::EXECUTE_DIRECT(unsigned, unsigned, Signal*, unsigned) (SimulatedBlock.hpp:722)
==915==    by 0x81766FB: Dblqh::nextScanConfLoopLab(Signal*) (DblqhMain.cpp:8252)
==915==    by 0x8176C5F: Dblqh::nextScanConfScanLab(Signal*) (DblqhMain.cpp:8186)
==915==    by 0x81933B8: Dblqh::execNEXT_SCANCONF(Signal*) (DblqhMain.cpp:6911)
==915==    by 0x80F342C: SimulatedBlock::executeFunction(unsigned short, Signal*) (SimulatedBlock.hpp:559)
==915==    by 0x80F35CF: SimulatedBlock::EXECUTE_DIRECT(unsigned, unsigned, Signal*, unsigned) (SimulatedBlock.hpp:722)
==915==    by 0x80EBD51: Dbacc::sendNextScanConf(Signal*) (DbaccMain.cpp:10295)
==915==    by 0x80EEC59: Dbacc::checkNextBucketLab(Signal*) (DbaccMain.cpp:9505)
==915==    by 0x80EF3AA: Dbacc::execACC_CHECK_SCAN(Signal*) (DbaccMain.cpp:9788)
==915==
==915== Conditional jump or move depends on uninitialised value(s)
==915==    at 0x81ECCE9: Dbtup::execTUPKEYREQ(Signal*) (DbtupExecQuery.cpp:771)
==915==    by 0x80F342C: SimulatedBlock::executeFunction(unsigned short, Signal*) (SimulatedBlock.hpp:559)
==915==    by 0x80F35CF: SimulatedBlock::EXECUTE_DIRECT(unsigned, unsigned, Signal*, unsigned) (SimulatedBlock.hpp:722)
==915==    by 0x8196C9D: Dblqh::execACCKEYCONF(Signal*) (DblqhMain.cpp:3985)
==915==    by 0x8197D2B: Dblqh::prepareContinueAfterBlockedLab(Signal*) (DblqhMain.cpp:3823)
==915==    by 0x81981DE: Dblqh::endgettupkeyLab(Signal*) (DblqhMain.cpp:3680)
==915==    by 0x8199475: Dblqh::execLQHKEYREQ(Signal*) (DblqhMain.cpp:3634)
==915==    by 0x80F342C: SimulatedBlock::executeFunction(unsigned short, Signal*) (SimulatedBlock.hpp:559)
==915==    by 0x827D96C: FastScheduler::doJob() (FastScheduler.cpp:137)
==915==    by 0x827ED93: ThreadConfig::ipControlLoop() (ThreadConfig.cpp:175)
==915==    by 0x80C1D3D: main (main.cpp:470)
==916==
==916== Thread 1:
==916== Conditional jump or move depends on uninitialised value(s)
==916==    at 0x81ECCE9: Dbtup::execTUPKEYREQ(Signal*) (DbtupExecQuery.cpp:771)
==916==    by 0x80F342C: SimulatedBlock::executeFunction(unsigned short, Signal*) (SimulatedBlock.hpp:559)
==916==    by 0x80F35CF: SimulatedBlock::EXECUTE_DIRECT(unsigned, unsigned, Signal*, unsigned) (SimulatedBlock.hpp:722)
==916==    by 0x8196C9D: Dblqh::execACCKEYCONF(Signal*) (DblqhMain.cpp:3985)
==916==    by 0x8197D2B: Dblqh::prepareContinueAfterBlockedLab(Signal*) (DblqhMain.cpp:3823)
==916==    by 0x81981DE: Dblqh::endgettupkeyLab(Signal*) (DblqhMain.cpp:3680)
==916==    by 0x8199475: Dblqh::execLQHKEYREQ(Signal*) (DblqhMain.cpp:3634)
==916==    by 0x80F342C: SimulatedBlock::executeFunction(unsigned short, Signal*) (SimulatedBlock.hpp:559)
==916==    by 0x827D96C: FastScheduler::doJob() (FastScheduler.cpp:137)
==916==    by 0x827ED93: ThreadConfig::ipControlLoop() (ThreadConfig.cpp:175)
==916==    by 0x80C1D3D: main (main.cpp:470)
==916==
==916== Conditional jump or move depends on uninitialised value(s)
==916==    at 0x81EBF5E: Dbtup::handleReadReq(Signal*, Dbtup::Operationrec*, Dbtup::Tablerec*, Dbtup::Page*) (DbtupExecQuery.cpp:873)
==916==    by 0x81ECA70: Dbtup::execTUPKEYREQ(Signal*) (DbtupExecQuery.cpp:722)
==916==    by 0x80F342C: SimulatedBlock::executeFunction(unsigned short, Signal*) (SimulatedBlock.hpp:559)
==916==    by 0x80F35CF: SimulatedBlock::EXECUTE_DIRECT(unsigned, unsigned, Signal*, unsigned) (SimulatedBlock.hpp:722)
==916==    by 0x81766FB: Dblqh::nextScanConfLoopLab(Signal*) (DblqhMain.cpp:8252)
==916==    by 0x8176C5F: Dblqh::nextScanConfScanLab(Signal*) (DblqhMain.cpp:8186)
==916==    by 0x81933B8: Dblqh::execNEXT_SCANCONF(Signal*) (DblqhMain.cpp:6911)
==916==    by 0x80F342C: SimulatedBlock::executeFunction(unsigned short, Signal*) (SimulatedBlock.hpp:559)
==916==    by 0x80F35CF: SimulatedBlock::EXECUTE_DIRECT(unsigned, unsigned, Signal*, unsigned) (SimulatedBlock.hpp:722)
==916==    by 0x80EBD51: Dbacc::sendNextScanConf(Signal*) (DbaccMain.cpp:10295)
==916==    by 0x80EEC59: Dbacc::checkNextBucketLab(Signal*) (DbaccMain.cpp:9505)
==916==    by 0x80EF3AA: Dbacc::execACC_CHECK_SCAN(Signal*) (DbaccMain.cpp:9788)

[23 May 2006 11:57] Stewart Smith

not showing up in pushbuild valgrind run, must have been fixed.