[FFmpeg-devel] [PATCH] avformat: added named pipe protocol

Michael Niedermayer michaelni at gmx.at
Fri Jun 15 02:52:53 CEST 2012


On Thu, Jun 14, 2012 at 01:10:30PM +0200, Martin Sliwka wrote:
> The new 'npipe' protocol is copy of 'file' protocol with two modifications:
> 1) URLProtocol::url_seek is not set
> 2) URLProtocol::url_open is set to npipe_open(...) which is same as
> file_open(...)
> except for protocol name it stripes from file name.
> 
> New protocol is also very similar to already included 'pipe' protocol
> which unfortunately can not be safely modified without breaking backward
> compatibility.
> 
> Reason for this change is that named pipes are not wokring as
> seekable inputs.
> Problem is that ffio_limit(...) incorrectly limits requested read
> size because
> avio_size(...) (it's fstat(...) in file_seek(...) in fact) can not
> report correct
> file size.

the correct solution is to check if its a fifo and return 0 instead of
the nonsense value from fstat()
a patch doing that is welcome

also thanks for the testcase

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

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- 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/20120615/4084997e/attachment.asc>


More information about the ffmpeg-devel mailing list