[Ffmpeg-devel] [PATCH] Fix for url_fclose() being called on an already closed file
Måns Rullgård
mru
Thu Feb 9 10:31:52 CET 2006
Colin Ward <lists at codehq.org> writes:
> M?ns Rullg?rd wrote:
>>
>>>>> While looking for a resource leak (which turned out to be in my own
>>>>> code!) I noticed that in libavformat/utils.c::av_open_input_file() it was
>>>>> possible for url_fclose() to be called twice in a row due to file_opened
>>>>> not being set back to 0 after the first call to url_fclose(). The chances
>>>>> of this are minutely tiny, but please find a patch attached anyway.
>>>> Could you set the mime type of attached patches to text/x-patch, please? It
>>>> makes them easier to read.
>>> Well, text/plain would be even prefered by me, if it works for you as
>>> well...
>> Sure, text/anything is better than application/binary. The advantage
>> of text/x-patch is that gnus will display it using diff-mode and
>> syntax highlighting. That makes the patches easier to read. Gnus
>> lets me display anything inline, but at work I have to use webmail and
>> it isn't as flexible.
>
> I had a look at my mailer and it seems that it adds attachments as
> application/octet-stream by default and there doesn't seem to be a
> way to change this. I agree that it is silly when you are
> attaching a text patch.
>
> I am going to make a feature request (its open source) and will
> attach future patches from my work computer using a different
> mailer.
Patches as application/octet-stream are better than no patches at
all. Don't delay things for that reason.
--
M?ns Rullg?rd
mru at inprovide.com
More information about the ffmpeg-devel
mailing list