[FFmpeg-cvslog] r12827 - trunk/ffmpeg.c

[Ramiro Polla]

Michael Niedermayer wrote:
> On Sun, Apr 20, 2008 at 05:04:37AM +0100, Ramiro Polla wrote:
>> ramiro wrote:
>>> Author: ramiro
>>> Date: Tue Apr 15 00:35:32 2008
>>> New Revision: 12827
>>> Log:
>>> Use av_exit() instead of exit() for a minimal cleanup
>>> before leaving ffmpeg.
>> Oops:
>>
>> ramiro at found:/usrc/ffmpeg/build$ ./ffmpeg_g -i ../data/video.mp4 -t 30 
>> output.ml20
>> FFmpeg version SVN-r12881, Copyright (c) 2000-2008 Fabrice Bellard, et al.
>>   configuration:
>>   libavutil version: 49.6.0
>>   libavcodec version: 51.55.0
>>   libavformat version: 52.13.0
>>   libavdevice version: 52.0.0
>>   built on Apr 17 2008 20:19:35, gcc: 4.1.3 20070929 (prerelease) (Ubuntu 
>> 4.1.2-16ubuntu2)
>> Input #0, mov,mp4,m4a,3gp,3g2,mj2, from '../data/video.mp4':
>>   Duration: 00:02:47.23, start: 0.000000, bitrate: 1333 kb/s
>>     Stream #0.0(eng): Video: mpeg4, yuv420p, 320x240 [PAR 1:1 DAR 4:3], 
>> 29.97 tb(r)
>>     Stream #0.1(eng): Data: rtp  / 0x20707472
>>     Stream #0.2(eng): Audio: mp4a / 0x6134706D, 44100 Hz, stereo
>>     Stream #0.3(eng): Data: rtp  / 0x20707472
>>     Stream #0.4(eng): Data: mp4s / 0x7334706D
>>     Stream #0.5(eng): Data: mp4s / 0x7334706D
>> File 'output.ml20' already exists. Overwrite ? [y/N] n
>> Not overwriting - exiting
>> Segmentation fault (core dumped)
>>
>> av_exit() tries to close a file that has not been opened (s->pb == NULL), 
>> and it segfaults in url_fclose().
>>
>> Attached patch fixes it.
> 
> ok

Applied.