[FFmpeg-devel] [RFC] av_tempfile()

Michael Niedermayer michaelni at gmx.at
Sun Oct 16 20:50:03 CEST 2011


On Sun, Oct 16, 2011 at 07:32:55PM +0200, Reimar Döffinger wrote:
> On Sun, Oct 16, 2011 at 03:31:07PM +0200, Michael Niedermayer wrote:
> > Hi
> > 
> > I think ff_tempfile() should be in libavutil as its a generically
> > usefull thing and i will also need it from libavformat. Currently
> > its only compiled when we link to xvid.
> > 
> > Ill move it to libavutil soon, but i wont update the version yet, only
> > once there is agreement (or lack of disagreement) on name, API and
> > place
> 
> I seem to remember some issues with it, so I wouldn't like too much for
> it become critical.
> The issues I see on a quick look are that the non-mkstemp fallback might
> result in overall code being unsafe, that it will always create the file
> in "." in that case.
> For the mkstemp code that it always tries /tmp which is not really
> appropriate for Windows,
> that it does not respect TEMPDIR or any of the
> other ones like that (note: I have more than once worked on systems
> where tmp was accessible but had performance that made it unusable),
> and it falls back to trying to create a file in "." for which I think
> there is more than one reason to dislike.
> Mostly I do not like of libav* creating any kind of file at all, ever,
> or if there is no alternative at all it should be something like mkstemp
> except that it will at also atomically unlink the file before returning
> its descriptor.

Some users might want to keep the temporary files for debuging or in
case of the cache protocol as a means to download while watching.

But what do you suggest how should it be implemented?
We have several features that depend on temporary files (the libxvid
wraper and the caching protocol)


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111016/5ff1e906/attachment.asc>


More information about the ffmpeg-devel mailing list