Merge commit 'e72605f80bf5cbe32053a554ccc137e0a99cf3dd'
authorMichael Niedermayer <michaelni@gmx.at>
Tue, 24 Feb 2015 21:42:00 +0000 (22:42 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Tue, 24 Feb 2015 21:42:04 +0000 (22:42 +0100)
* commit 'e72605f80bf5cbe32053a554ccc137e0a99cf3dd':
  rtpdec: Allow allocating and freeing the private data without explicit functions

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavformat/rtpdec.h
libavformat/rtsp.c

diff --combined libavformat/rtpdec.h
@@@ -3,20 -3,20 +3,20 @@@
   * Copyright (c) 2002 Fabrice Bellard
   * Copyright (c) 2006 Ryan Martell <rdm4@martellventures.com>
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -120,6 -120,7 +120,7 @@@ struct RTPDynamicProtocolHandler 
      int static_payload_id; /* 0 means no payload id is set. 0 is a valid
                              * payload ID (PCMU), too, but that format doesn't
                              * require any custom depacketization code. */
+     int priv_data_size;
  
      /** Initialize dynamic protocol handler, called after the full rtpmap line is parsed, may be null */
      int (*init)(AVFormatContext *s, int st_index, PayloadContext *priv_data);
@@@ -174,9 -175,9 +175,9 @@@ struct RTPDemuxContext 
      /*@}*/
  
      /* rtcp sender statistics receive */
 -    int64_t last_rtcp_ntp_time;
 +    uint64_t last_rtcp_ntp_time;
      int64_t last_rtcp_reception_time;
 -    int64_t first_rtcp_ntp_time;
 +    uint64_t first_rtcp_ntp_time;
      uint32_t last_rtcp_timestamp;
      int64_t rtcp_ts_offset;
  
diff --combined libavformat/rtsp.c
@@@ -2,24 -2,23 +2,24 @@@
   * RTSP/SDP client
   * Copyright (c) 2002 Fabrice Bellard
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
 +#include "libavutil/avassert.h"
  #include "libavutil/base64.h"
  #include "libavutil/avstring.h"
  #include "libavutil/intreadwrite.h"
  
  #define RTSP_FLAG_OPTS(name, longname) \
      { name, longname, OFFSET(rtsp_flags), AV_OPT_TYPE_FLAGS, {.i64 = 0}, INT_MIN, INT_MAX, DEC, "rtsp_flags" }, \
 -    { "filter_src", "Only receive packets from the negotiated peer IP", 0, AV_OPT_TYPE_CONST, {.i64 = RTSP_FLAG_FILTER_SRC}, 0, 0, DEC, "rtsp_flags" }
 +    { "filter_src", "only receive packets from the negotiated peer IP", 0, AV_OPT_TYPE_CONST, {.i64 = RTSP_FLAG_FILTER_SRC}, 0, 0, DEC, "rtsp_flags" }
  
  #define RTSP_MEDIATYPE_OPTS(name, longname) \
 -    { name, longname, OFFSET(media_type_mask), AV_OPT_TYPE_FLAGS, { .i64 = (1 << (AVMEDIA_TYPE_DATA+1)) - 1 }, INT_MIN, INT_MAX, DEC, "allowed_media_types" }, \
 +    { name, longname, OFFSET(media_type_mask), AV_OPT_TYPE_FLAGS, { .i64 = (1 << (AVMEDIA_TYPE_SUBTITLE+1)) - 1 }, INT_MIN, INT_MAX, DEC, "allowed_media_types" }, \
      { "video", "Video", 0, AV_OPT_TYPE_CONST, {.i64 = 1 << AVMEDIA_TYPE_VIDEO}, 0, 0, DEC, "allowed_media_types" }, \
      { "audio", "Audio", 0, AV_OPT_TYPE_CONST, {.i64 = 1 << AVMEDIA_TYPE_AUDIO}, 0, 0, DEC, "allowed_media_types" }, \
 -    { "data", "Data", 0, AV_OPT_TYPE_CONST, {.i64 = 1 << AVMEDIA_TYPE_DATA}, 0, 0, DEC, "allowed_media_types" }
 +    { "data", "Data", 0, AV_OPT_TYPE_CONST, {.i64 = 1 << AVMEDIA_TYPE_DATA}, 0, 0, DEC, "allowed_media_types" }, \
 +    { "subtitle", "Subtitle", 0, AV_OPT_TYPE_CONST, {.i64 = 1 << AVMEDIA_TYPE_SUBTITLE}, 0, 0, DEC, "allowed_media_types" }
  
  #define RTSP_REORDERING_OPTS() \
 -    { "reorder_queue_size", "Number of packets to buffer for handling of reordered packets", OFFSET(reordering_queue_size), AV_OPT_TYPE_INT, { .i64 = -1 }, -1, INT_MAX, DEC }
 +    { "reorder_queue_size", "set number of packets to buffer for handling of reordered packets", OFFSET(reordering_queue_size), AV_OPT_TYPE_INT, { .i64 = -1 }, -1, INT_MAX, DEC }
  
  const AVOption ff_rtsp_options[] = {
 -    { "initial_pause",  "Don't start playing the stream immediately", OFFSET(initial_pause), AV_OPT_TYPE_INT, {.i64 = 0}, 0, 1, DEC },
 +    { "initial_pause",  "do not start playing the stream immediately", OFFSET(initial_pause), AV_OPT_TYPE_INT, {.i64 = 0}, 0, 1, DEC },
      FF_RTP_FLAG_OPTS(RTSPState, rtp_muxer_flags),
 -    { "rtsp_transport", "RTSP transport protocols", OFFSET(lower_transport_mask), AV_OPT_TYPE_FLAGS, {.i64 = 0}, INT_MIN, INT_MAX, DEC|ENC, "rtsp_transport" }, \
 +    { "rtsp_transport", "set RTSP transport protocols", OFFSET(lower_transport_mask), AV_OPT_TYPE_FLAGS, {.i64 = 0}, INT_MIN, INT_MAX, DEC|ENC, "rtsp_transport" }, \
      { "udp", "UDP", 0, AV_OPT_TYPE_CONST, {.i64 = 1 << RTSP_LOWER_TRANSPORT_UDP}, 0, 0, DEC|ENC, "rtsp_transport" }, \
      { "tcp", "TCP", 0, AV_OPT_TYPE_CONST, {.i64 = 1 << RTSP_LOWER_TRANSPORT_TCP}, 0, 0, DEC|ENC, "rtsp_transport" }, \
      { "udp_multicast", "UDP multicast", 0, AV_OPT_TYPE_CONST, {.i64 = 1 << RTSP_LOWER_TRANSPORT_UDP_MULTICAST}, 0, 0, DEC, "rtsp_transport" },
      { "http", "HTTP tunneling", 0, AV_OPT_TYPE_CONST, {.i64 = (1 << RTSP_LOWER_TRANSPORT_HTTP)}, 0, 0, DEC, "rtsp_transport" },
 -    RTSP_FLAG_OPTS("rtsp_flags", "RTSP flags"),
 -    { "listen", "Wait for incoming connections", 0, AV_OPT_TYPE_CONST, {.i64 = RTSP_FLAG_LISTEN}, 0, 0, DEC, "rtsp_flags" },
 -    RTSP_MEDIATYPE_OPTS("allowed_media_types", "Media types to accept from the server"),
 -    { "min_port", "Minimum local UDP port", OFFSET(rtp_port_min), AV_OPT_TYPE_INT, {.i64 = RTSP_RTP_PORT_MIN}, 0, 65535, DEC|ENC },
 -    { "max_port", "Maximum local UDP port", OFFSET(rtp_port_max), AV_OPT_TYPE_INT, {.i64 = RTSP_RTP_PORT_MAX}, 0, 65535, DEC|ENC },
 -    { "timeout", "Maximum timeout (in seconds) to wait for incoming connections. -1 is infinite. Implies flag listen", OFFSET(initial_timeout), AV_OPT_TYPE_INT, {.i64 = -1}, INT_MIN, INT_MAX, DEC },
 +    RTSP_FLAG_OPTS("rtsp_flags", "set RTSP flags"),
 +    { "listen", "wait for incoming connections", 0, AV_OPT_TYPE_CONST, {.i64 = RTSP_FLAG_LISTEN}, 0, 0, DEC, "rtsp_flags" },
 +    { "prefer_tcp", "try RTP via TCP first, if available", 0, AV_OPT_TYPE_CONST, {.i64 = RTSP_FLAG_PREFER_TCP}, 0, 0, DEC|ENC, "rtsp_flags" },
 +    RTSP_MEDIATYPE_OPTS("allowed_media_types", "set media types to accept from the server"),
 +    { "min_port", "set minimum local UDP port", OFFSET(rtp_port_min), AV_OPT_TYPE_INT, {.i64 = RTSP_RTP_PORT_MIN}, 0, 65535, DEC|ENC },
 +    { "max_port", "set maximum local UDP port", OFFSET(rtp_port_max), AV_OPT_TYPE_INT, {.i64 = RTSP_RTP_PORT_MAX}, 0, 65535, DEC|ENC },
 +    { "timeout", "set maximum timeout (in seconds) to wait for incoming connections (-1 is infinite, imply flag listen)", OFFSET(initial_timeout), AV_OPT_TYPE_INT, {.i64 = -1}, INT_MIN, INT_MAX, DEC },
 +    { "stimeout", "set timeout (in microseconds) of socket TCP I/O operations", OFFSET(stimeout), AV_OPT_TYPE_INT, {.i64 = 0}, INT_MIN, INT_MAX, DEC },
      RTSP_REORDERING_OPTS(),
 +    { "user-agent", "override User-Agent header", OFFSET(user_agent), AV_OPT_TYPE_STRING, {.str = LIBAVFORMAT_IDENT}, 0, 0, DEC },
      { NULL },
  };
  
  static const AVOption sdp_options[] = {
      RTSP_FLAG_OPTS("sdp_flags", "SDP flags"),
 -    { "custom_io", "Use custom IO", 0, AV_OPT_TYPE_CONST, {.i64 = RTSP_FLAG_CUSTOM_IO}, 0, 0, DEC, "rtsp_flags" },
 -    { "rtcp_to_source", "Send RTCP packets to the source address of received packets", 0, AV_OPT_TYPE_CONST, {.i64 = RTSP_FLAG_RTCP_TO_SOURCE}, 0, 0, DEC, "rtsp_flags" },
 -    RTSP_MEDIATYPE_OPTS("allowed_media_types", "Media types to accept from the server"),
 +    { "custom_io", "use custom I/O", 0, AV_OPT_TYPE_CONST, {.i64 = RTSP_FLAG_CUSTOM_IO}, 0, 0, DEC, "rtsp_flags" },
 +    { "rtcp_to_source", "send RTCP packets to the source address of received packets", 0, AV_OPT_TYPE_CONST, {.i64 = RTSP_FLAG_RTCP_TO_SOURCE}, 0, 0, DEC, "rtsp_flags" },
 +    RTSP_MEDIATYPE_OPTS("allowed_media_types", "set media types to accept from the server"),
      RTSP_REORDERING_OPTS(),
      { NULL },
  };
  
  static const AVOption rtp_options[] = {
 -    RTSP_FLAG_OPTS("rtp_flags", "RTP flags"),
 +    RTSP_FLAG_OPTS("rtp_flags", "set RTP flags"),
      RTSP_REORDERING_OPTS(),
      { NULL },
  };
@@@ -195,6 -190,10 +195,10 @@@ static void init_rtp_handler(RTPDynamic
          rtsp_st->dynamic_protocol_context = handler->alloc();
          if (!rtsp_st->dynamic_protocol_context)
              rtsp_st->dynamic_handler = NULL;
+     } else if (handler->priv_data_size) {
+         rtsp_st->dynamic_protocol_context = av_mallocz(handler->priv_data_size);
+         if (!rtsp_st->dynamic_protocol_context)
+             rtsp_st->dynamic_handler = NULL;
      }
  }
  
@@@ -403,8 -402,6 +407,8 @@@ static void sdp_parse_line(AVFormatCont
              codec_type = AVMEDIA_TYPE_VIDEO;
          } else if (!strcmp(st_type, "application") || !strcmp(st_type, "text")) {
              codec_type = AVMEDIA_TYPE_DATA;
 +        } else if (!strcmp(st_type, "text")) {
 +            codec_type = AVMEDIA_TYPE_SUBTITLE;
          }
          if (codec_type == AVMEDIA_TYPE_UNKNOWN || !(rt->media_type_mask & (1 << codec_type))) {
              s1->skip_media = 1;
              /* no corresponding stream */
              if (rt->transport == RTSP_TRANSPORT_RAW) {
                  if (CONFIG_RTPDEC && !rt->ts)
 -                    rt->ts = ff_mpegts_parse_open(s);
 +                    rt->ts = avpriv_mpegts_parse_open(s);
              } else {
                  RTPDynamicProtocolHandler *handler;
                  handler = ff_rtp_handler_find_by_id(
@@@ -670,13 -667,13 +674,13 @@@ int ff_sdp_parse(AVFormatContext *s, co
      }
  
      for (i = 0; i < s1->nb_default_include_source_addrs; i++)
 -        av_free(s1->default_include_source_addrs[i]);
 +        av_freep(&s1->default_include_source_addrs[i]);
      av_freep(&s1->default_include_source_addrs);
      for (i = 0; i < s1->nb_default_exclude_source_addrs; i++)
 -        av_free(s1->default_exclude_source_addrs[i]);
 +        av_freep(&s1->default_exclude_source_addrs[i]);
      av_freep(&s1->default_exclude_source_addrs);
  
 -    rt->p = av_malloc(sizeof(struct pollfd)*2*(rt->nb_rtsp_streams+1));
 +    rt->p = av_malloc_array(rt->nb_rtsp_streams + 1, sizeof(struct pollfd) * 2);
      if (!rt->p) return AVERROR(ENOMEM);
      return 0;
  }
@@@ -702,7 -699,7 +706,7 @@@ void ff_rtsp_undo_setup(AVFormatContex
                      avio_close_dyn_buf(rtpctx->pb, &ptr);
                      av_free(ptr);
                  } else {
 -                    avio_close(rtpctx->pb);
 +                    avio_closep(&rtpctx->pb);
                  }
                  avformat_free_context(rtpctx);
              } else if (CONFIG_RTPDEC && rt->transport == RTSP_TRANSPORT_RDT)
@@@ -728,27 -725,31 +732,31 @@@ void ff_rtsp_close_streams(AVFormatCont
      for (i = 0; i < rt->nb_rtsp_streams; i++) {
          rtsp_st = rt->rtsp_streams[i];
          if (rtsp_st) {
-             if (rtsp_st->dynamic_handler && rtsp_st->dynamic_protocol_context)
-                 rtsp_st->dynamic_handler->free(
-                     rtsp_st->dynamic_protocol_context);
+             if (rtsp_st->dynamic_handler && rtsp_st->dynamic_protocol_context) {
+                 if (rtsp_st->dynamic_handler->free)
+                     rtsp_st->dynamic_handler->free(
+                         rtsp_st->dynamic_protocol_context);
+                 else
+                     av_free(rtsp_st->dynamic_protocol_context);
+             }
              for (j = 0; j < rtsp_st->nb_include_source_addrs; j++)
 -                av_free(rtsp_st->include_source_addrs[j]);
 +                av_freep(&rtsp_st->include_source_addrs[j]);
              av_freep(&rtsp_st->include_source_addrs);
              for (j = 0; j < rtsp_st->nb_exclude_source_addrs; j++)
 -                av_free(rtsp_st->exclude_source_addrs[j]);
 +                av_freep(&rtsp_st->exclude_source_addrs[j]);
              av_freep(&rtsp_st->exclude_source_addrs);
  
 -            av_free(rtsp_st);
 +            av_freep(&rtsp_st);
          }
      }
 -    av_free(rt->rtsp_streams);
 +    av_freep(&rt->rtsp_streams);
      if (rt->asf_ctx) {
          avformat_close_input(&rt->asf_ctx);
      }
      if (CONFIG_RTPDEC && rt->ts)
 -        ff_mpegts_parse_close(rt->ts);
 -    av_free(rt->p);
 -    av_free(rt->recvbuf);
 +        avpriv_mpegts_parse_close(rt->ts);
 +    av_freep(&rt->p);
 +    av_freep(&rt->recvbuf);
  }
  
  int ff_rtsp_open_transport_ctx(AVFormatContext *s, RTSPStream *rtsp_st)
@@@ -1176,7 -1177,7 +1184,7 @@@ start
      if (content_ptr)
          *content_ptr = content;
      else
 -        av_free(content);
 +        av_freep(&content);
  
      if (request) {
          char buf[1024];
@@@ -1265,7 -1266,7 +1273,7 @@@ static int rtsp_send_cmd_with_content_a
      if (headers)
          av_strlcat(buf, headers, sizeof(buf));
      av_strlcatf(buf, sizeof(buf), "CSeq: %d\r\n", rt->seq);
 -    av_strlcatf(buf, sizeof(buf), "User-Agent: %s\r\n", LIBAVFORMAT_IDENT);
 +    av_strlcatf(buf, sizeof(buf), "User-Agent: %s\r\n",  rt->user_agent);
      if (rt->session_id[0] != '\0' && (!headers ||
          !strstr(headers, "\nIf-Match:"))) {
          av_strlcatf(buf, sizeof(buf), "Session: %s\r\n", rt->session_id);
@@@ -1376,6 -1377,10 +1384,6 @@@ int ff_rtsp_make_setup_request(AVFormat
      /* default timeout: 1 minute */
      rt->timeout = 60;
  
 -    /* for each stream, make the setup request */
 -    /* XXX: we assume the same server is used for the control of each
 -     * RTSP stream */
 -
      /* Choose a random starting offset within the first half of the
       * port range, to allow for a number of ports to try even if the offset
       * happens to be at the end of the random range. */
                                 &s->interrupt_callback, NULL))
                      goto rtp_opened;
              }
 -
              av_log(s, AV_LOG_ERROR, "Unable to open an input RTP port\n");
              err = AVERROR(EIO);
              goto fail;
              goto fail;
          } else if (reply->status_code != RTSP_STATUS_OK ||
                     reply->nb_transports != 1) {
 -            err = AVERROR_INVALIDDATA;
 +            err = ff_rtsp_averror(reply->status_code, AVERROR_INVALIDDATA);
              goto fail;
          }
  
@@@ -1748,14 -1754,12 +1756,14 @@@ redirect
              goto fail;
          }
      } else {
 +        int ret;
          /* open the tcp connection */
          ff_url_join(tcpname, sizeof(tcpname), lower_rtsp_proto, NULL,
 -                    host, port, NULL);
 -        if (ffurl_open(&rt->rtsp_hd, tcpname, AVIO_FLAG_READ_WRITE,
 -                       &s->interrupt_callback, NULL) < 0) {
 -            err = AVERROR(EIO);
 +                    host, port,
 +                    "?timeout=%d", rt->stimeout);
 +        if ((ret = ffurl_open(&rt->rtsp_hd, tcpname, AVIO_FLAG_READ_WRITE,
 +                       &s->interrupt_callback, NULL)) < 0) {
 +            err = ret;
              goto fail;
          }
          rt->rtsp_hd_out = rt->rtsp_hd;
                         sizeof(cmd));
          ff_rtsp_send_cmd(s, "OPTIONS", rt->control_uri, cmd, reply, NULL);
          if (reply->status_code != RTSP_STATUS_OK) {
 -            err = AVERROR_INVALIDDATA;
 +            err = ff_rtsp_averror(reply->status_code, AVERROR_INVALIDDATA);
              goto fail;
          }
  
          err = ff_rtsp_setup_input_streams(s, reply);
      else if (CONFIG_RTSP_MUXER)
          err = ff_rtsp_setup_output_streams(s, host);
 +    else
 +        av_assert0(0);
      if (err)
          goto fail;
  
          int lower_transport = ff_log2_tab[lower_transport_mask &
                                    ~(lower_transport_mask - 1)];
  
 +        if ((lower_transport_mask & (1 << RTSP_LOWER_TRANSPORT_TCP))
 +                && (rt->rtsp_flags & RTSP_FLAG_PREFER_TCP))
 +            lower_transport = RTSP_LOWER_TRANSPORT_TCP;
 +
          err = ff_rtsp_make_setup_request(s, host, port, lower_transport,
                                   rt->server_type == RTSP_SERVER_REAL ?
                                       real_challenge : NULL);
@@@ -1900,7 -1898,7 +1908,7 @@@ static int udp_read_packet(AVFormatCont
                      p[max_p].fd       = fds[fdsidx];
                      p[max_p++].events = POLLIN;
                  }
 -                av_free(fds);
 +                av_freep(&fds);
              }
          }
          n = poll(p, max_p, POLL_TIMEOUT_MS);
@@@ -2010,7 -2008,7 +2018,7 @@@ int ff_rtsp_fetch_packet(AVFormatContex
          } else if (rt->transport == RTSP_TRANSPORT_RTP) {
              ret = ff_rtp_parse_packet(rt->cur_transport_priv, pkt, NULL, 0);
          } else if (CONFIG_RTPDEC && rt->ts) {
 -            ret = ff_mpegts_parse_packet(rt->ts, pkt, rt->recvbuf + rt->recvbuf_pos, rt->recvbuf_len - rt->recvbuf_pos);
 +            ret = avpriv_mpegts_parse_packet(rt->ts, pkt, rt->recvbuf + rt->recvbuf_pos, rt->recvbuf_len - rt->recvbuf_pos);
              if (ret >= 0) {
                  rt->recvbuf_pos += ret;
                  ret = rt->recvbuf_pos < rt->recvbuf_len;
@@@ -2127,16 -2125,6 +2135,16 @@@ redo
                              st2->time_base);
                      }
                  }
 +                // Make real NTP start time available in AVFormatContext
 +                if (s->start_time_realtime == AV_NOPTS_VALUE) {
 +                    s->start_time_realtime = av_rescale (rtpctx->first_rtcp_ntp_time - (NTP_OFFSET << 32), 1000000, 1LL << 32);
 +                    if (rtpctx->st) {
 +                        s->start_time_realtime -=
 +                            av_rescale (rtpctx->rtcp_ts_offset,
 +                                        (uint64_t) rtpctx->st->time_base.num * 1000000,
 +                                                   rtpctx->st->time_base.den);
 +                    }
 +                }
              }
              if (ret == -RTCP_BYE) {
                  rt->nb_byes++;
              }
          }
      } else if (CONFIG_RTPDEC && rt->ts) {
 -        ret = ff_mpegts_parse_packet(rt->ts, pkt, rt->recvbuf, len);
 +        ret = avpriv_mpegts_parse_packet(rt->ts, pkt, rt->recvbuf, len);
          if (ret >= 0) {
              if (ret < len) {
                  rt->recvbuf_len = len;
@@@ -2232,7 -2220,7 +2240,7 @@@ static int sdp_read_header(AVFormatCont
      content[size] ='\0';
  
      err = ff_sdp_parse(s, content);
 -    av_free(content);
 +    av_freep(&content);
      if (err) goto fail;
  
      /* open each RTP stream */
@@@ -2383,7 -2371,7 +2391,7 @@@ static int rtp_read_header(AVFormatCont
      /* sdp_read_header initializes this again */
      ff_network_close();
  
 -    rt->media_type_mask = (1 << (AVMEDIA_TYPE_DATA+1)) - 1;
 +    rt->media_type_mask = (1 << (AVMEDIA_TYPE_SUBTITLE+1)) - 1;
  
      ret = sdp_read_header(s);
      s->pb = NULL;