[FFmpeg-devel] [PATCH v4] avformat/libsrt: print streamid at client

["zhilizhao(赵志立)"]

> On Jul 23, 2021, at 5:59 PM, Raghavendra Rao Sidlagatta <raghavendra.rao at collabora.com> wrote:
> 
> 
> On Monday, March 22, 2021 06:45 GMT, "Raghavendra Rao Sidlagatta" <raghavendra.rao at collabora.com> wrote:
>   
> On Wednesday, January 20, 2021 09:48 GMT, "Raghavendra Rao Sidlagatta" <raghavendra.rao at collabora.com> wrote:
>   
> On Tuesday, October 06, 2020 08:18 BST, raghavendra <raghavendra.rao at collabora.com> wrote:
>  Print the SRT streamid at the client.
> Logged to info.
> 
> Signed-off-by: raghavendra <raghavendra.rao at collabora.com>
> ---
> libavformat/libsrt.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
> 
> diff --git a/libavformat/libsrt.c b/libavformat/libsrt.c
> index 4025b24976..eed48c11cf 100644
> --- a/libavformat/libsrt.c
> +++ b/libavformat/libsrt.c
> @@ -359,6 +359,13 @@ static int libsrt_set_options_pre(URLContext *h, int fd)
> return 0;
> }
> 
> +static void libsrt_dump_streamid(URLContext *h, int fd)
> +{
> + char streamid[512];
> + int optlen = sizeof(streamid);
> + if (!libsrt_getsockopt(h, fd, SRTO_STREAMID, "SRTO_STREAMID", streamid, &optlen))
> + av_log(h, AV_LOG_INFO, "srt_streamid : %s\n", streamid);
> +}

Sorry I didn’t notice your patch and have sent a similar one:

https://patchwork.ffmpeg.org/project/ffmpeg/patch/tencent_AE6202C169367CFAACEE39D9C15DB4A8690A@qq.com/

Please note that the array length of streamid should be 513. From the doc:
----
	• string - a C string. When setting an option, a const char* character array pointer is expected to be passed in optval and the array length in optlen without the terminating NULL character. When getting, an array is expected to be passed in optval with a sufficient size with an extra space for the terminating NULL character provided in optlen. The return value of optlen does not count the terminating NULL character.
----


> 
> static int libsrt_setup(URLContext *h, const char *uri, int flags)
> {
> @@ -442,6 +449,8 @@ static int libsrt_setup(URLContext *h, const char *uri, int flags)
> goto fail1;
> listen_fd = fd;
> fd = ret;
> + // dump srt streamid at client
> + libsrt_dump_streamid(h, fd);
> } else {
> if (s->mode == SRT_MODE_RENDEZVOUS) {
> ret = srt_bind(fd, cur_ai->ai_addr, cur_ai->ai_addrlen);
> --
> 2.25.1
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> 
> Hello There,
> 
> Could you please let me know the upstream status of this patch?
> 
> Best Regards,
> Ragahvendra
>  
> 
> 
> Hello There,
> 
> Could you please let me know the upstream status of this patch?
> 
> Best Regards,
> Ragahvendra
> 
> Hello There,
> 
> Could you please let me know the upstream status of this patch?
> 
> Best Regards,
> Ragahvendra.
> 
>  
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".