diff mbox series

[FFmpeg-devel,v1] avformat/rtsp: RTP support buffer_size&pkt_size option

Message ID ME3P282MB18127E0B7F52387E3AF7C31CC7929@ME3P282MB1812.AUSP282.PROD.OUTLOOK.COM
State New
Headers show
Series [FFmpeg-devel,v1] avformat/rtsp: RTP support buffer_size&pkt_size option
Related show

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

gaojiangjie@live.com March 9, 2021, 6:21 a.m. UTC
From: Jiangjie Gao <gaojiangjie@live.com>

And forward it to udp.
issue: https://trac.ffmpeg.org/ticket/7517

Signed-off-by: Jiangjie Gao <gaojiangjie@live.com>
---
 libavformat/rtsp.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comments

Marton Balint March 10, 2021, 8:27 p.m. UTC | #1
On Tue, 9 Mar 2021, gaojiangjie@live.com wrote:

> From: Jiangjie Gao <gaojiangjie@live.com>
>
> And forward it to udp.
> issue: https://trac.ffmpeg.org/ticket/7517

Could you add the buffer_size option to the documentation of the 
RTP protocol?

Thanks,
Marton

>
> Signed-off-by: Jiangjie Gao <gaojiangjie@live.com>
> ---
> libavformat/rtsp.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
> index 9a2933346e..25bdf475b3 100644
> --- a/libavformat/rtsp.c
> +++ b/libavformat/rtsp.c
> @@ -2506,12 +2506,15 @@ static int rtp_read_header(AVFormatContext *s)
>     RTSPState *rt = s->priv_data;
>     const char *p;
>     AVBPrint sdp;
> +    AVDictionary *opts = NULL;
>
>     if (!ff_network_init())
>         return AVERROR(EIO);
> 
> +    opts = map_to_opts(rt);
>     ret = ffurl_open_whitelist(&in, s->url, AVIO_FLAG_READ,
> -                     &s->interrupt_callback, NULL, s->protocol_whitelist, s->protocol_blacklist, NULL);
> +                     &s->interrupt_callback, &opts, s->protocol_whitelist, s->protocol_blacklist, NULL);
> +    av_dict_free(&opts);
>     if (ret)
>         goto fail;
> 
> -- 
> 2.27.0.windows.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
Marton Balint March 19, 2021, 10:20 p.m. UTC | #2
On Wed, 10 Mar 2021, Marton Balint wrote:

>
>
> On Tue, 9 Mar 2021, gaojiangjie@live.com wrote:
>
>> From: Jiangjie Gao <gaojiangjie@live.com>
>>
>> And forward it to udp.
>> issue: https://trac.ffmpeg.org/ticket/7517
>
> Could you add the buffer_size option to the documentation of the 
> RTP protocol?

Did it, and applied.

Thanks,
Marton

>>
>> Signed-off-by: Jiangjie Gao <gaojiangjie@live.com>
>> ---
>> libavformat/rtsp.c | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
>> index 9a2933346e..25bdf475b3 100644
>> --- a/libavformat/rtsp.c
>> +++ b/libavformat/rtsp.c
>> @@ -2506,12 +2506,15 @@ static int rtp_read_header(AVFormatContext *s)
>>     RTSPState *rt = s->priv_data;
>>     const char *p;
>>     AVBPrint sdp;
>> +    AVDictionary *opts = NULL;
>>
>>     if (!ff_network_init())
>>         return AVERROR(EIO);
>> 
>> +    opts = map_to_opts(rt);
>>     ret = ffurl_open_whitelist(&in, s->url, AVIO_FLAG_READ,
>> -                     &s->interrupt_callback, NULL, s->protocol_whitelist, 
> s->protocol_blacklist, NULL);
>> +                     &s->interrupt_callback, &opts, s->protocol_whitelist, 
> s->protocol_blacklist, NULL);
>> +    av_dict_free(&opts);
>>     if (ret)
>>         goto fail;
>> 
>> -- 
>> 2.27.0.windows.1
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel@ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
diff mbox series

Patch

diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
index 9a2933346e..25bdf475b3 100644
--- a/libavformat/rtsp.c
+++ b/libavformat/rtsp.c
@@ -2506,12 +2506,15 @@  static int rtp_read_header(AVFormatContext *s)
     RTSPState *rt = s->priv_data;
     const char *p;
     AVBPrint sdp;
+    AVDictionary *opts = NULL;
 
     if (!ff_network_init())
         return AVERROR(EIO);
 
+    opts = map_to_opts(rt);
     ret = ffurl_open_whitelist(&in, s->url, AVIO_FLAG_READ,
-                     &s->interrupt_callback, NULL, s->protocol_whitelist, s->protocol_blacklist, NULL);
+                     &s->interrupt_callback, &opts, s->protocol_whitelist, s->protocol_blacklist, NULL);
+    av_dict_free(&opts);
     if (ret)
         goto fail;