[Decode] Added copy range check for BDW

Fixes #537

Signed-off-by: Dmitry Ermilov <dmitry.ermilov@intel.com>

Author: dmitryermilov

media_driver/agnostic/common/codec/hal/codechal_decode_mpeg2.cpp

@@ -218,10 +218,13 @@ MOS_STATUS CodechalDecodeMpeg2::CopyDataSurface(
             "Failed to allocate copied residual data buffer.");
     }
 
-    // initialize lock flags
-    MOS_LOCK_PARAMS lockFlagsWriteOnly;
-    MOS_ZeroMemory(&lockFlagsWriteOnly, sizeof(MOS_LOCK_PARAMS));
-    lockFlagsWriteOnly.WriteOnly = 1;
+    if ((m_nextCopiedDataOffset + dataSize) > m_copiedDataBufferSize)
+    {
+        CODECHAL_DECODE_ASSERTMESSAGE("Copied data buffer is not large enough.");
+
+        m_slicesInvalid = true;
+        return MOS_STATUS_UNKNOWN;
+    }
 
     uint32_t size = MOS_ALIGN_CEIL(dataSize, 16); // 16 byte aligned
 
@@ -245,14 +248,6 @@ MOS_STATUS CodechalDecodeMpeg2::CopyDataSurface(
         return MOS_STATUS_SUCCESS;
     }
 
-    if ((m_nextCopiedDataOffset + dataSize) > m_copiedDataBufferSize)
-    {
-        CODECHAL_DECODE_ASSERTMESSAGE("Copied data buffer is not large enough.");
-
-        m_slicesInvalid = true;
-        return MOS_STATUS_UNKNOWN;
-    }
-
     CODECHAL_DECODE_CHK_STATUS_RETURN(m_osInterface->pfnSetGpuContext(
         m_osInterface,
         m_videoContextForWa));