av_reallocp_array does the check already.
---
 libavformat/mxfdec.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
index ea5ba58..e22b92a 100644
--- a/libavformat/mxfdec.c
+++ b/libavformat/mxfdec.c
@@ -418,9 +418,6 @@ static int mxf_read_partition_pack(void *arg, AVIOContext 
*pb, int tag, int size
     uint32_t nb_essence_containers;
     int err;
 
-    if (mxf->partitions_count+1 >= UINT_MAX / sizeof(*mxf->partitions))
-        return AVERROR(ENOMEM);
-
     if ((err = av_reallocp_array(&mxf->partitions, mxf->partitions_count + 1,
                                  sizeof(*mxf->partitions))) < 0) {
         mxf->partitions_count = 0;
@@ -552,8 +549,6 @@ static int mxf_add_metadata_set(MXFContext *mxf, void 
*metadata_set)
 {
     int err;
 
-    if (mxf->metadata_sets_count+1 >= UINT_MAX / sizeof(*mxf->metadata_sets))
-        return AVERROR(ENOMEM);
     if ((err = av_reallocp_array(&mxf->metadata_sets, mxf->metadata_sets_count 
+ 1,
                                  sizeof(*mxf->metadata_sets))) < 0) {
         mxf->metadata_sets_count = 0;
-- 
1.8.5.1

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to