kvs_stream: pxDataFrame used before input validation (#91)

Author: ak1932

src/source/stream/stream.c

@@ -460,12 +460,6 @@ int Kvs_dataFrameAddTags(DataFrameHandle xDataFrameHandle, MkvTag_t* tagsList, s
 
     static bool firstClusterSeen = false;
 
-    if (pxDataFrame->xDataFrameIn.xClusterType != MKV_CLUSTER)
-    {
-        LogInfo("Not a cluster");
-        return KVS_ERRNO_NONE;
-    }
-
     // Input validation
     if (pxDataFrame == NULL || ppMkvHeader == NULL ||
         puMkvHeaderLen == NULL || ppData == NULL || puDataLen == NULL)
@@ -474,6 +468,12 @@ int Kvs_dataFrameAddTags(DataFrameHandle xDataFrameHandle, MkvTag_t* tagsList, s
         return KVS_ERROR_INVALID_ARGUMENT;
     }
 
+    if (pxDataFrame->xDataFrameIn.xClusterType != MKV_CLUSTER)
+    {
+        LogInfo("Not a cluster");
+        return KVS_ERRNO_NONE;
+    }
+
     size_t effectiveTagsLen = tagsListLen + (endOfStream ? 1 : 0);
     if (effectiveTagsLen >= MAX_TAG_AMOUNT) {
         LogError("Trying to add too many tags - Adding: %zu tags, Max: %d", effectiveTagsLen, MAX_TAG_AMOUNT);