[FFmpeg-devel] [PATCH] lavf/mxfdec.c: fix MXF essence body offsets when only 1 body partition

Jason Livingston jason at cpcweb.com
Tue Mar 19 18:49:37 CET 2013


On Monday, March 18, 2013 4:55 AM, ffmpeg-devel-bounces at ffmpeg.org [ffmpeg-devel-bounces at ffmpeg.org] on behalf of Tomas H?rdin [tomas.hardin at codemill.se] wrote:
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH] lavf/mxfdec.c: fix MXF essence body offsets when only 1 body partition
> 
> On Wed, 2013-03-06 at 16:37 +0000, Jason Livingston wrote:
>> I was having trouble with some Avid OP-Atom MXF files that contain only 1 essence track and only 1 body partition with all the frames.  After the first call to av_read_frame(), it would return AVERROR_INVALIDDATA from mxf_absolute_bodysid_offset():
>> [mxf @ xxx] failed to find absolute offset of 100 in BodySID 1 - partial file?
>>
>> The attached patch fixes this case which is probably pretty rare, but it shouldn't negatively affect anything else.
> 
> I dunno, the patch looks a little suspicious. Most files I work with do
> indeed only have one body partition, so this is probably not the real
> issue. I'll try and have a closer look later today.
> 
> /Tomas

Hi Tomas, thanks for taking a look.  It's quite possible my description of the problem was incorrect, or maybe my patch worked by accident.  Either way I'd certainly like to see this fixed so please let me know what you find out.

Thanks again,
Jason


More information about the ffmpeg-devel mailing list