Merge pull request #785 from daijh/fix-scan-issues

Fix code scan issues

Author: daijh

source/core/owt_base/LiveStreamIn.cpp

@@ -69,9 +69,10 @@ FramePacket::FramePacket (AVPacket *packet)
     : m_packet(NULL)
 {
     m_packet = (AVPacket *)malloc(sizeof(AVPacket));
-
-    av_init_packet(m_packet);
-    av_packet_ref(m_packet, packet);
+    if (m_packet) {
+      av_init_packet(m_packet);
+      av_packet_ref(m_packet, packet);
+    }
 }
 
 FramePacket::~FramePacket()
@@ -1153,6 +1154,10 @@ bool LiveStreamIn::parse_avcC(AVPacket *pkt) {
 
                 nals_buf_length += nalsize + 4;
                 nals_buf = (uint8_t *)realloc(nals_buf, nals_buf_length);
+                if (nals_buf == nullptr) {
+                    ELOG_ERROR_T("OOM! Allocate size %d", nals_buf_length);
+                    return false;
+                }
             }
 
             nals_buf[nals_size] = 0;

source/core/owt_base/MsdkFrameDecoder.cpp

@@ -466,9 +466,13 @@ void MsdkFrameDecoder::updateBitstream(const Frame& frame)
     }
 
     if(m_bitstream->DataOffset + m_bitstream->DataLength + frame.length > m_bitstream->MaxLength) {
-        memmove(m_bitstream->Data, m_bitstream->Data + m_bitstream->DataOffset, m_bitstream->DataLength);
-        m_bitstream->DataOffset = 0;
-        ELOG_TRACE_T("Move bitstream buffer offset");
+        if (m_bitstream->Data) {
+            memmove(m_bitstream->Data, m_bitstream->Data + m_bitstream->DataOffset, m_bitstream->DataLength);
+            m_bitstream->DataOffset = 0;
+            ELOG_TRACE_T("Move bitstream buffer offset");
+        } else { // make code scanner happy
+            return;
+        }
     }
 
     memcpy(m_bitstream->Data + m_bitstream->DataOffset + m_bitstream->DataLength, frame.payload, frame.length);