Opened 3 years ago

Closed 3 years ago

#289 closed defect (fixed)

Accessing uninitialized memory

Reported by: ksuehring Owned by:
Priority: minor Milestone: VTM-5.1
Component: VTM Version: VTM-5.0
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…

Description (last modified by ksuehring)

When running the encoder in valgrind, I get the following error during encoding of the first frame:

==96718== Conditional jump or move depends on uninitialised value(s)
==96718==    at 0x1002CE4FB: CABACWriter::codeAlfCtuEnableFlag(CodingStructure&, unsigned int, int, AlfSliceParam*) (CABACWriter.cpp:3456)
==96718==    by 0x1002CDDB4: CABACWriter::coding_tree_unit(CodingStructure&, UnitArea const&, int (&) [2], unsigned int, bool) (CABACWriter.cpp:172)
==96718==    by 0x1003B84A6: EncSlice::encodeCtus(Picture*, bool, bool, unsigned int, unsigned int, EncLib*) (EncSlice.cpp:1773)
==96718==    by 0x1003B5EFF: EncSlice::compressSlice(Picture*, bool, bool) (EncSlice.cpp:1480)
==96718==    by 0x100337355: EncGOP::compressGOP(int, int, std::__1::list<Picture*, std::__1::allocator<Picture*> >&, std::__1::list<UnitBuf<short>*, std::__1::allocator<UnitBuf<short>*> >&, bool, bool, InputColourSpaceConversion, bool, bool) (EncGOP.cpp:2331)
==96718==    by 0x10036A75D: EncLib::encode(bool, PelStorage*, PelStorage*, InputColourSpaceConversion, std::__1::list<UnitBuf<short>*, std::__1::allocator<UnitBuf<short>*> >&, int&) (EncLib.cpp:657)
==96718==    by 0x10000A02A: EncApp::encode() (EncApp.cpp:767)
==96718==    by 0x1000577DA: main (encmain.cpp:153)
==96718== 
==96718== Conditional jump or move depends on uninitialised value(s)
==96718==    at 0x1002CE899: CABACWriter::codeAlfCtuFilterIndex(CodingStructure&, unsigned int, bool) (CABACWriter.cpp:3591)
==96718==    by 0x1002CDE2B: CABACWriter::coding_tree_unit(CodingStructure&, UnitArea const&, int (&) [2], unsigned int, bool) (CABACWriter.cpp:176)
==96718==    by 0x1003B84A6: EncSlice::encodeCtus(Picture*, bool, bool, unsigned int, unsigned int, EncLib*) (EncSlice.cpp:1773)
==96718==    by 0x1003B5EFF: EncSlice::compressSlice(Picture*, bool, bool) (EncSlice.cpp:1480)
==96718==    by 0x100337355: EncGOP::compressGOP(int, int, std::__1::list<Picture*, std::__1::allocator<Picture*> >&, std::__1::list<UnitBuf<short>*, std::__1::allocator<UnitBuf<short>*> >&, bool, bool, InputColourSpaceConversion, bool, bool) (EncGOP.cpp:2331)
==96718==    by 0x10036A75D: EncLib::encode(bool, PelStorage*, PelStorage*, InputColourSpaceConversion, std::__1::list<UnitBuf<short>*, std::__1::allocator<UnitBuf<short>*> >&, int&) (EncLib.cpp:657)
==96718==    by 0x10000A02A: EncApp::encode() (EncApp.cpp:767)
==96718==    by 0x1000577DA: main (encmain.cpp:153)
==96718== 

Steps to reproduce:

git clone https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM.git
git checkout f84a8bc5059bb27dce70120727fbe317bc65121b
make debug
valgrind <path_to>/EncoderApp -c cfg/per-sequence/RaceHorses.cfg -c cfg/encoder_randomaccess_vtm.cfg -q 37 -ipp <source path> -f 16

May be related to #281

Change history (3)

comment:1 Changed 3 years ago by ksuehring

  • Description modified (diff)

comment:2 Changed 3 years ago by nanhu

A merge request (https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/571) has been submitted. Could you check if the warning is still there?

comment:3 Changed 3 years ago by ksuehring

  • Milestone set to VTM-5.1
  • Resolution set to fixed
  • Status changed from new to closed

The problems are resolved.

Note: See TracTickets for help on using tickets.