]> git.sesse.net Git - ffmpeg/commitdiff
avformat/argo_brp: remove an allocation
authorZane van Iperen <zane@zanevaniperen.com>
Sat, 19 Sep 2020 12:02:42 +0000 (22:02 +1000)
committerZane van Iperen <zane@zanevaniperen.com>
Sun, 20 Sep 2020 09:21:00 +0000 (19:21 +1000)
Signed-off-by: Zane van Iperen <zane@zanevaniperen.com>
libavformat/argo_brp.c

index 27029d07b1ae5e3b2d3a15aab4adbe7a827ad86d..d6f7eb237adcf146164333bc35f657fe7a9bf574 100644 (file)
@@ -77,7 +77,7 @@ typedef struct ArgoBRPStreamHeader {
 
 typedef struct ArgoBRPDemuxContext {
     ArgoBRPFileHeader   fhdr;
-    ArgoBRPStreamHeader *streams;
+    ArgoBRPStreamHeader streams[BRP_MAX_STREAMS];
     /* To know how much of a BASF to give. */
     int64_t             lastpts;
     int                 hit_eof;
@@ -101,16 +101,6 @@ static int argo_brp_probe(const AVProbeData *p)
     return AVPROBE_SCORE_EXTENSION + 1;
 }
 
-static int argo_brp_read_close(AVFormatContext *s)
-{
-    ArgoBRPDemuxContext *brp = s->priv_data;
-
-    if (brp->streams != NULL)
-        av_freep(&brp->streams);
-
-    return 0;
-}
-
 static int read_extradata(AVFormatContext *s, const ArgoBRPStreamHeader *hdr,
                           void *buf, size_t bufsz)
 {
@@ -174,9 +164,6 @@ static int argo_brp_read_header(AVFormatContext *s)
         return AVERROR_PATCHWELCOME;
     }
 
-    if ((brp->streams = av_mallocz_array(brp->fhdr.num_streams, sizeof(ArgoBRPStreamHeader))) == NULL)
-        return AVERROR(ENOMEM);
-
     /* Build the stream info. */
     brp->basf.index = -1;
     for (uint32_t i = 0; i < brp->fhdr.num_streams; i++) {
@@ -331,8 +318,6 @@ static int argo_brp_read_header(AVFormatContext *s)
     return 0;
 
 fail:
-    /* TODO: Remove once AVFMT_HEADER_CLEANUP lands. */
-    argo_brp_read_close(s);
     return ret;
 }
 
@@ -445,5 +430,4 @@ AVInputFormat ff_argo_brp_demuxer = {
     .read_probe     = argo_brp_probe,
     .read_header    = argo_brp_read_header,
     .read_packet    = argo_brp_read_packet,
-    .read_close     = argo_brp_read_close
 };