[Decode] dctbl pointer check code refactor

chuanli1 · dmitryermilov · commit 83dd5e013c5d · 2022-02-25T11:06:05.000+03:00
Issue: MDP-68544
Test: manual
diff --git a/_studio/shared/umc/codec/jpeg_dec/src/jpegdec.cpp b/_studio/shared/umc/codec/jpeg_dec/src/jpegdec.cpp
@@ -2376,10 +2376,10 @@ JERRCODE CJPEGDecoder::DecodeHuffmanMCURowBL(int16_t* pMCUBuf, uint32_t colMCU,
       {
           return JPEG_ERR_PARAMS;
       }
+      if (m_dctbl[m_ccomp[n].m_dc_selector].IsEmpty())
+          return JPEG_ERR_PARAMS;      
       IppiDecodeHuffmanSpec* dctbl  = m_dctbl[m_ccomp[n].m_dc_selector];
       IppiDecodeHuffmanSpec* actbl  = m_actbl[m_ccomp[n].m_ac_selector];
-      if(dctbl == nullptr)
-        return JPEG_ERR_PARAMS;
       for(k = 0; k < m_ccomp[n].m_scan_vsampling; k++)
       {
         for(l = 0; l < m_ccomp[n].m_scan_hsampling; l++)

Original file line number	Diff line number	Diff line change
`@@ -2376,10 +2376,10 @@ JERRCODE CJPEGDecoder::DecodeHuffmanMCURowBL(int16_t* pMCUBuf, uint32_t colMCU,`
`2376`	`2376`	`{`
`2377`	`2377`	`return JPEG_ERR_PARAMS;`
`2378`	`2378`	`}`
	`2379`	`+ if (m_dctbl[m_ccomp[n].m_dc_selector].IsEmpty())`
	`2380`	`+ return JPEG_ERR_PARAMS;`
`2379`	`2381`	`IppiDecodeHuffmanSpec* dctbl = m_dctbl[m_ccomp[n].m_dc_selector];`
`2380`	`2382`	`IppiDecodeHuffmanSpec* actbl = m_actbl[m_ccomp[n].m_ac_selector];`
`2381`		`- if(dctbl == nullptr)`
`2382`		`- return JPEG_ERR_PARAMS;`
`2383`	`2383`	`for(k = 0; k < m_ccomp[n].m_scan_vsampling; k++)`
`2384`	`2384`	`{`
`2385`	`2385`	`for(l = 0; l < m_ccomp[n].m_scan_hsampling; l++)`