[Ffmpeg-devel] [PATCH] Fix for url_fclose() being called on an already closed file

[Colin Ward]

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.

-- 
/-------------------------------------------------------------------\
[Hitman/Code HQ - 6502/z80/68000/604e/80x86/ARM coder - Amiga rulez!]
[VZ-200/VIC-20/MZ-700/c16/c64*10/c128*8/Plus-4/CPC464/CD32/500*2    ]
[600/1000/1200*2/A4000/SNES/N64/Dreamcast/Athlon 1100/AmigaOne      ]
[Assembly Language: The most fun you can have with your clothes on! ]
\-------------------------------------------------------------------/