FFmpeg

Commit Graph

Author	SHA1	Message	Date
Andreas Rheinhardt	233e13f285	avformat/mux: Rename FF_FMT_ALLOW_FLUSH->FF_OFMT_FLAG_ALLOW_FLUSH It better reflects that this is a muxer-only flag. Also document the flag. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	8 months ago
Andreas Rheinhardt	41ab25cb2d	avformat/fifo_test: Move into tests/fifo_muxer.c This muxer solely exists to test the fifo muxer via a dedicated test tool in libavformat/tests/fifo_muxer.c. It fulfills no other role and it is only designed with this role in mind. The latter can be seen in two facts: The muxer uses printf for logging and it simply presumes the packets' data to contain a FailingMuxerPacketData (a struct duplicated in fifo_test.c and tests/fifo_muxer.c.); in particular, it presumes packets to have data at all, but this need not be true with side-data only packets and a segfault can easily be triggered by e.g. encoding flac (our native encoder sends a side-data only packet with updated extradata at the end of encoding). This patch fixes this by moving the test muxer into the fifo test tool, making it inaccessible via the API (and actually removing it from libavformat.so and libavformat.a). While this muxer was accessible via e.g. av_guess_format(), it was not really usable for an API user as FailingMuxerPacketData was not public. Therefore this is not considered a breaking change. In order to continue to use the test muxer in the test tool, the ordinary fifo muxer had to be overridden: fifo_muxer.c includes lavf/fifo.c but with FIFO_TEST defined which makes it support the fifo_test muxer. This is possible because test tools are always linked statically to their respective library. Reviewed-by: Stefano Sabatini <stefasab@gmail.com> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	8 months ago
Andreas Rheinhardt	2934a4b9a5	Remove unnecessary avassert.h inclusions Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	3 years ago
Andreas Rheinhardt	edcbb3e1b9	avformat/tests/fifo_muxer: Fix memleak on error, fix API violation The test program for the FIFO muxer allocates a buffer without padding and wraps it into a packet via av_packet_from_data(). This is an API violation. Furthermore, said buffer leaks in case av_packet_from_data() fails. Fix both of these issues by using av_new_packet() instead. Reviewed-by: James Almer <jamrial@gmail.com> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>	4 years ago
Andreas Rheinhardt	aff1d373b1	avformat/tests/fifo_muxer: Fix leak of AVPacket on error Also factor allocating and freeing the packet out. Fixes Coverity issues #1473722 and #1473723; it is a regression since `4b386b2059`. Reviewed-by: James Almer <jamrial@gmail.com> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>	4 years ago
James Almer	4b386b2059	avformat/tests/fifo_muxer: use av_packet_alloc() to allocate packets Signed-off-by: James Almer <jamrial@gmail.com>	4 years ago
Josh de Kock	598d5f8579	lavf: move fifo test muxer into separate file This fixes the fate-fifo-muxer test with the clarified removal of registering external formats.	7 years ago
James Almer	d6d605eb05	avformat/mux: stop delaying writing the header Every bitstream filter behaves as intended now, so there's no need to wait for the first packet of every stream. Signed-off-by: James Almer <jamrial@gmail.com>	7 years ago
Steven Liu	d1f3e475f9	avformat/test/fifo_muxer: add check for FailingMuxerPacketData alloc CID: 1396257 Reviewed-by: Michael Niedermayer <michael@niedermayer.cc> Signed-off-by: Steven Liu <lq@chinaffmpeg.org>	8 years ago
Gregory J. Wolfe	9c041a3cd5	avformat/tests/fifo_muxer: includes libavformat/network.h to define ETIMEDOUT for fate build. Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>	8 years ago
Jan Sebechlebsky	b84c83144d	avformat/fifo: Add fate test Tested-by: Michael Niedermayer <michael@niedermayer.cc> Signed-off-by: Jan Sebechlebsky <sebechlebskyjan@gmail.com> Signed-off-by: Marton Balint <cus@passwd.hu>	8 years ago

11 Commits (e93793bf3cf15968c34b0e7bf0c677fad3032f5d)