[FFmpeg-devel] [PATCH] flvdec: Check the avio_seek return value after reading a metadata packet

2023-06-20 Thread Armstrong Huang
In most cases, flv_read_metabody reads pass the beginning of the meta_pos. If the beginning of the meta_pos had been flushed from the IO buffer, we would not be able to seek to the right position (for a nonseekable stream). Is better to check the seek result and skip the current flv body if

[FFmpeg-devel] [PATCH] flvdec: Check the avio_seek return value after reading a metadata packet

2023-06-19 Thread Armstrong Huang
In most cases, flv_read_metabody reads pass the beginning of the meta_pos. if the beginning of the meta_pos had been flushed from the IO buffer, we would not be able to seek to the right position (for a nonseekable stream). Is better to check the seek result and skip the current flv body if

Re: [FFmpeg-devel] [PATCH] flvdec: Check the avio_seek return value after reading a metadata packet

2017-10-26 Thread Carl Eugen Hoyos
2017-09-05 18:38 GMT+02:00 Steven Liu : > COPY FROM libav Martin Storsjö This should have been the author information of the patch instead of part of the commit message, you removed it completely from the actual commit;-( Imo, the best solution is that

Re: [FFmpeg-devel] [PATCH] flvdec: Check the avio_seek return value after reading a metadata packet

2017-09-06 Thread Steven Liu
2017-09-06 0:38 GMT+08:00 Steven Liu : > COPY FROM libav Martin Storsjö > > If the metadata packet is corrupted, flv_read_metabody can accidentally > read past the start of the next packet. If the start of the next packet > had been flushed out of the IO

[FFmpeg-devel] [PATCH] flvdec: Check the avio_seek return value after reading a metadata packet

2017-09-05 Thread Steven Liu
COPY FROM libav Martin Storsjö If the metadata packet is corrupted, flv_read_metabody can accidentally read past the start of the next packet. If the start of the next packet had been flushed out of the IO buffer, we would be unable to seek to the right position (on a