From 40aa27b788c0b13250312d3f876406b27229a91a Mon Sep 17 00:00:00 2001 From: Luis de Bethencourt Date: Wed, 18 Nov 2015 16:01:48 +0000 Subject: [PATCH] matroskamux: remove duplicate check We want 1 or 2 streamheaders, the check if (bufarr->len != 1 && bufarr->len != 2) is enough. Not need to check if bufarr->len is <= 0 or > 255. --- gst/matroska/matroska-mux.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/gst/matroska/matroska-mux.c b/gst/matroska/matroska-mux.c index d60b86341c..2f2d762b6e 100644 --- a/gst/matroska/matroska-mux.c +++ b/gst/matroska/matroska-mux.c @@ -1616,9 +1616,7 @@ opus_streamheader_to_codecdata (const GValue * streamheader, goto wrong_type; bufarr = g_value_peek_pointer (streamheader); - if (bufarr->len <= 0 || bufarr->len > 255) /* one header, and count stored in a byte */ - goto wrong_count; - if (bufarr->len != 1 && bufarr->len != 2) + if (bufarr->len != 1 && bufarr->len != 2) /* one header, and count stored in a byte */ goto wrong_count; /* Opus headers are not in-band */