Hi,
* Nico Golde <[email protected]> [2008-11-21 17:30]:
[...] 
> Looking at the code I think this is just a null ptr dereference:
> 1062     if(!is_b8x8)
> 1063         *mb_type |= MB_TYPE_DIRECT2;
> 1064     if(MB_FIELD)
> 1065         *mb_type |= MB_TYPE_INTERLACED;
> 1066 
> 1067     tprintf(s->avctx, "mb_type = %08x, sub_mb_type = %08x, is_b8x8 = %d, 
> mb_type_col = %08x\n", *mb_type, sub_mb_type, is_b8x8, mb_type_col);
> 
> is_b8x8 is checked before but not when dereferencing it in tprintf.

This is of course wrong as I overlooked that this will be 
casted to an integer.

Cheers
Nico
-- 
Nico Golde - http://www.ngolde.de - [email protected] - GPG: 0x73647CFF
For security reasons, all text in this mail is double-rot13 encrypted.

Attachment: pgpjigMsGDH8B.pgp
Description: PGP signature

Reply via email to