[FFmpeg-devel] Indeo3 maintainer?

[Reimar Döffinger]

On Mon, Jul 06, 2009 at 05:26:05AM +0200, Michael Niedermayer wrote:
> On Sun, Jul 05, 2009 at 02:28:41PM +0200, Reimar D?ffinger wrote:
> > am I right that we have no maintainer for indeo3 (only Michael in his
> > "fallback" role)?
> 
> too much is falling back on me, iam failing misserably to maintain code
> that is mine to maintain let alone suff like the obfuscated indeo3 ...

I sure wasn't trying to push this on you!

> > The problem seems to me that the code lacks buffer boundary checks all
> 
> i remembr that .... 
> if you think there are missing checks on writng instead of just reading then
> iam fine with disabling indeo3 until maxim or someone else can maintain it
> properly

Well, reading the code I concluded the issue is mostly (only?) with reference
frames, in which case it should only be reads.
Unfortunately valgrind says otherwise, which is no surprise with code
like that:
                cp2 = s->ModPred + ((lv - 8) << 7);
                cp = ref_frm_pos;
                for(i = 0; i < blks_width << 2; i++) {
                    int v = *cp >> 1;
                    *(cp++) = cp2[v];
                }

I assumed that ref_frm_pos would be reference frame, but here it is
written into.
So if nobody can say more about it I think it might be better to disable
it by default, I think we've usually done that by commenting out in
allcodecs.c, right?