Opened 13 months ago

Closed 13 months ago

#1592 closed enhancement (fixed)

fix m_olsHrdParams in SPS class to avoid crash

Reported by: marie-pierre.gallasso Owned by:
Priority: critical Milestone:
Component: VTM Version: VTM-19.0
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…

Description

since commit 0992b1a773f448135841e9394f263a663c2f0c71 m_olsHrdParams variable member of SPS class, is no longer an array.
But in VLCReader.cpp, in parseOlsHrdParameters function, there is a loop on maxNumSubLayersMinus1 to fill m_olsHrdParams. It seems that m_olsHrdParams can contain more than one entry.

void HLSyntaxReader::parseOlsHrdParameters(GeneralHrdParams * generalHrd, OlsHrdParams *olsHrd, uint32_t firstSubLayer, uint32_t maxNumSubLayersMinus1)
{
  uint32_t  symbol;

  for( int i = firstSubLayer; i <= maxNumSubLayersMinus1; i ++ )
  {
     OlsHrdParams *hrd = &(olsHrd[i]);
     ...
  }
}

Please modify the length of m_olsHrdParams to avoid crash.

Change history (2)

comment:1 Changed 13 months ago by marie-pierre.gallasso

fix in commit ac04ba2c2c366f2e6334054d3da8b52e8f2a5900

comment:2 Changed 13 months ago by marie-pierre.gallasso

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.