[FFmpeg-cvslog] lavf/segment: remove duplicated and inconsistent cleanup code in seg_write_packet()

Stefano Sabatini git at videolan.org
Fri Mar 13 17:48:59 CET 2015


ffmpeg | branch: release/1.1 | Stefano Sabatini <stefasab at gmail.com> | Tue Jan 21 19:58:41 2014 +0100| [e76a5a9c284c033210485b952ec4cadc05034503] | committer: Michael Niedermayer

lavf/segment: remove duplicated and inconsistent cleanup code in seg_write_packet()

In particular, avoid to leave around the seg->avf pointer to freed
structure, and fix crash with:
ffmpeg -f lavfi -i testsrc -c:v h264 -map 0 -f segment foo-%d.ts
(cherry picked from commit 169065fbfb3da1ab776379c333aebc54bb1f1bc4)

Found-by: Qinghao Tang
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=e76a5a9c284c033210485b952ec4cadc05034503
---

 libavformat/segment.c |    6 ------
 1 file changed, 6 deletions(-)

diff --git a/libavformat/segment.c b/libavformat/segment.c
index efc35c8..76616f8 100644
--- a/libavformat/segment.c
+++ b/libavformat/segment.c
@@ -656,12 +656,6 @@ fail:
     if (pkt->stream_index == seg->reference_stream_index)
         seg->frame_count++;
 
-    if (ret < 0) {
-        if (seg->list)
-            avio_close(seg->list_pb);
-        avformat_free_context(oc);
-    }
-
     return ret;
 }
 



More information about the ffmpeg-cvslog mailing list