[FFmpeg-devel] [PATCH] Turn off usage of base_data_offset in tfhd for PIFF files

[Michael Niedermayer]

On Thu, May 16, 2013 at 10:24:39PM +0200, Michael Niedermayer wrote:
> On Thu, May 16, 2013 at 06:57:47PM +0200, Jeremy Hinegardner wrote:
> > Hi all
> > 
> > I am generating ISMV files via ffmpeg, and the player we are using is quite
> > strict according to the PIFF specification. 
> > 
> > It appears that when ffmpeg is creating ISMV files it is setting the 
> > `base_data_offset` flag which is contrary to what the PIFF specification[1] specifies. 
> > Section 5.2.17 says "the base_data_offset field MUST be omitted".
> > 
> > I managed to track this down and it appears the fix is just a single bit flag to 
> > be blanked out. With this patch the players we have tested are now happy
> > with the ismv files created by ffmpeg.
> > 
> > thanks for you time,
> > 
> > [1] http://www.iis.net/learn/media/smooth-streaming/protected-interoperable-file-format
> 
> the patch breaks
> make fate-lavf-ismv
> 
> Please take a look at the generated file and if ok update the checksum
> for fate-lavf-ismv with the patch that also changes movenc.c

checksums updated, patch applied

Feel free to double check the generated file if you have some means
to do so

Thsanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- 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/20130517/faa2df1d/attachment.asc>