[FFmpeg-devel] [PATCH] rmdec.c: correctly skip indexes

[Kostya]

On Mon, Mar 09, 2009 at 11:39:00AM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On Mon, Mar 9, 2009 at 11:07 AM, Kostya <kostya.shishkov at gmail.com> wrote:
> > We have some docs:
> > https://common.helixcommunity.org/2003/HCS_SDK_r5/htmfiles/rmff.htm
> 
> Ah, fancy, I've looked for that for a while but couldn't find it (I
> did find the RDT docs - very useful).
> 
> > I suspect it was not widely available and is violated though.
> 
> The funny thing is that it's violated on-and-off in one and the same
> file. Unfortunately, no "producer" or anything in it, so I don't know
> what software was made to make these files. It's clearly warez-style
> crap so not officially .rm, but it'd be nice if we could play it back
> correctly anyway.

That's our goal - to play anything and starting with ".rm" - with rmdec.c :|
 
> Oh, and if the header minimal size is 0x14 + 14 bytes for one index
> chunk, I guess the >=28 in my patch should be changed into
> >=(0x14+14)?

I prefer it to be that way:
if header_size == 0x14 and num_indices > 0 then calculate real size (and skip)
otherwise check header_size value to be 0x14+num_indices*0xE and throw
a warning if it is not (and skip that size anyway).
 
> Ronald