From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id F24B34779A for ; Wed, 22 Nov 2023 09:33:16 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 9084568CDBE; Wed, 22 Nov 2023 11:33:13 +0200 (EET) Received: from mail8.parnet.fi (mail8.parnet.fi [77.234.108.134]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 3370068CBAE for ; Wed, 22 Nov 2023 11:33:07 +0200 (EET) Received: from mail9.parnet.fi (mail9.parnet.fi [77.234.108.21]) by mail8.parnet.fi with ESMTP id 3AM9X5KM003761-3AM9X5KN003761 for ; Wed, 22 Nov 2023 11:33:05 +0200 Received: from foo.martin.st (host-97-144.parnet.fi [77.234.97.144]) by mail9.parnet.fi (Postfix) with ESMTPS id D1641A1478 for ; Wed, 22 Nov 2023 11:33:05 +0200 (EET) Date: Wed, 22 Nov 2023 11:33:05 +0200 (EET) From: =?ISO-8859-15?Q?Martin_Storsj=F6?= To: FFmpeg development discussions and patches In-Reply-To: Message-ID: References: MIME-Version: 1.0 X-FE-Policy-ID: 3:14:2:SYSTEM Subject: Re: [FFmpeg-devel] [PATCH] avformat/rtmpproto: Pass rw_timeout to underlying transport protocol X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Wed, 22 Nov 2023, Zhao Zhili wrote: > > >> On Nov 15, 2023, at 21:24, Zhao Zhili wrote: >> >> From: Zhao Zhili >> >> Signed-off-by: Zhao Zhili >> --- >> libavformat/rtmpproto.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c >> index 98718bc6da..a0c6195eb2 100644 >> --- a/libavformat/rtmpproto.c >> +++ b/libavformat/rtmpproto.c >> @@ -2635,6 +2635,9 @@ static int rtmp_open(URLContext *s, const char *uri, int flags, AVDictionary **o >> >> if (rt->listen_timeout > 0) >> rt->listen = 1; >> + /* Pass rw_timeout to underlying transport protocol */ >> + if (s->rw_timeout > 0) >> + av_dict_set_int(opts, "rw_timeout", s->rw_timeout, 0); > > OK, I made a mistake. Since ffurl_open_whitelist copy from parent automatically, > this is a NOP. Will revert this commit. Thanks! I was just starting to look into this, to confirm this as well, but it's good if you've come to the same conclusion already. It's interesting how we end up with similar changes for this multiple times, see https://ffmpeg.org/pipermail/ffmpeg-devel/2023-July/312362.html for another similar patch a few months ago, when it should be working already. The fact that so many protocols have similar but vaguely different timeout options, each defined as a per-protocol private option, is a bit of a mess; that's why this approach, of actually sharing the same variable through the URLContext hierarchy, tries to avoid that to some extent. // Martin _______________________________________________ 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".