2014-09-11 10:29 GMT+11:00 wm4 <[email protected]>:

> On Thu, 11 Sep 2014 09:58:28 +1100
> Alexander Drozdov <[email protected]> wrote:
>
> > Ughr... ffmpeg-devel: not for bug reports... It is evil that I can't
> simple
> > send patch/report via mail :-(
>
> This is just so that ffmpeg-devel is not flooded with user questions or
> support requests. Sending a patch to ffmpeg-devel is always ok.
>

I prepare patch with git format-patch, but I can't send it to ffmpeg-devel,
link https://lists.ffmpeg.org/mailman/listinfo/ffmpeg-devel/ unavail now. I
can't subscribe.

Patch is attached.

-- 
WBR, Alexander Drozdov
http://htrd.su
From f742012ea22bf684fe0ae8e153749bd7339503d6 Mon Sep 17 00:00:00 2001
From: Alexander Drozdov <[email protected]>
Date: Fri, 26 Sep 2014 09:45:08 +1100
Subject: [PATCH] RTMP: fix FD leak in rtmp_open()

If we setup AVIO interrupt callback and it will be returns 1 on socket
timeouts and we try to connect to non-existing streams on some servers
(like nginx-rtmp) we got FD leak.
---
 libavformat/rtmpproto.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c
index 3cde966..6122548 100644
--- a/libavformat/rtmpproto.c
+++ b/libavformat/rtmpproto.c
@@ -2670,7 +2670,7 @@ reconnect:
         // audio or video packet arrives.
         while (!rt->has_audio && !rt->has_video && !rt->received_metadata) {
             if ((ret = get_packet(s, 0)) < 0)
-               return ret;
+               goto fail;
         }
 
         // Either after we have read the metadata or (if there is none) the
-- 
1.9.1

_______________________________________________
Libav-user mailing list
[email protected]
http://ffmpeg.org/mailman/listinfo/libav-user

Reply via email to