shithub: openh264

Info  •  Files  •  Log  •  Branches

Merge pull request #153 from licaiguo/fix-empty-packets

fix crash caused by empty packets and add more checks -- review request #44

--- a/codec/decoder/core/src/au_parser.cpp

+++ b/codec/decoder/core/src/au_parser.cpp

@@ -479,7 +479,8 @@

   int32_t iPicHeight		= 0;

   int32_t iBitSize		= 0;

   int32_t iErr				= ERR_NONE;

-

+  if (kiSrcLen <= 0)

+    return iErr;

   pBs	     = &pCtx->sBs;	// SBitStringAux instance for non VCL NALs decoding

   iBitSize = (kiSrcLen << 3) - BsGetTrailingBits (pRbsp + kiSrcLen - 1); // convert into bit

   eNalType = pCtx->sCurNalHead.eNalUnitType;

--- a/codec/decoder/core/src/decode_slice.cpp

+++ b/codec/decoder/core/src/decode_slice.cpp

@@ -403,7 +403,7 @@

   iNextMbXyIndex = pSliceHeader->iFirstMbInSlice;

-  if (iNextMbXyIndex >= kiCountNumMb) {

+  if ((iNextMbXyIndex < 0) || (iNextMbXyIndex >= kiCountNumMb)) {

     WelsLog (pCtx, WELS_LOG_ERROR,

              "WelsDecodeSlice()::iFirstMbInSlice(%d) > pSps->kiTotalMb(%d). ERROR!!! resolution change....\n",

              iNextMbXyIndex, kiCountNumMb);

--- a/codec/decoder/core/src/decoder.cpp

+++ b/codec/decoder/core/src/decoder.cpp

@@ -690,7 +690,7 @@

  */

 void_t UpdateMaxPictureResolution (PWelsDecoderContext pCtx, const int32_t kiCurWidth, const int32_t kiCurHeight) {

   //any dimension larger than that of current dimension, should modify the max-dimension

-  if (kiCurWidth > pCtx->iMaxWidthInSps || kiCurHeight > pCtx->iMaxHeightInSps) {

+  if ((kiCurWidth * kiCurHeight) > (pCtx->iMaxWidthInSps * pCtx->iMaxHeightInSps)) {

     pCtx->iMaxWidthInSps	= kiCurWidth;

     pCtx->iMaxHeightInSps	= kiCurHeight;

   }

-- 

⑨