[FFmpeg-devel] av_read_frame timeout

2019-11-17 Thread gga
The following patch adds a timeout interrupt to av_read_frame to prevent it from hanging up the application.  This patch was proposed some years ago but was not applied back then.  I believe it is useful and should be considered for approval. diff --git a/libavformat/utils.c b/libavformat/util

Re: [FFmpeg-devel] av_read_frame timeout

2019-11-18 Thread gga
El 18/11/19 a las 14:12, Michael Niedermayer escribió: On Sun, Nov 17, 2019 at 06:37:58PM -0300, gga wrote: The following patch adds a timeout interrupt to av_read_frame to prevent it from hanging up the application.  This patch was proposed some years ago but was not applied back then.  I

Re: [FFmpeg-devel] [PATCH] avformat/utils.c: allows av_read_frame to return after a timeout period.

2019-11-24 Thread gga
Friendly ping.  Does the patch need anything else for approval? On 21/11/19 18:27, ggarr...@gmail.com wrote: From: Gonzalo Garramuño This patch is based on a patch by bsenftner at earthlink.net. --- libavformat/utils.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/uti

Re: [FFmpeg-devel] [PATCH] avformat/utils.c: allows av_read_frame to return after a timeout period.

2019-11-26 Thread gga
On 26/11/19 14:31, Michael Niedermayer wrote: On Thu, Nov 21, 2019 at 06:27:10PM -0300, ggarr...@gmail.com wrote: From: Gonzalo Garramuño This patch is based on a patch by bsenftner at earthlink.net. --- libavformat/utils.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavfor

Re: [FFmpeg-devel] [PATCH] avformat/utils.c: allows av_read_frame to return after a timeout period.

2019-11-29 Thread gga
On 29/11/19 15:35, Michael Niedermayer wrote: also, do we have a testcase where this loop hangs ? if not please provide a testcase I don't have an easy testcase, but it can be seen with any rtmp stream.  A full test case can be reproduced like this: Compile and create a nginx server like it i

Re: [FFmpeg-devel] [PATCH] avformat/utils.c: allows av_read_frame to return after a timeout period.

2019-12-02 Thread gga
On 1/12/19 09:05, Andreas Rheinhardt wrote: ggarr...@gmail.com: From: Gonzalo Garramuño Moved the check inside and at the end of the if (pktl) as per Michael Niedermayer's suggestion. This patch is based on one from Blake Senftner ( bsenftner at earthlink.net ). The hanging in av_read_frame