[MPlayer-dev-eng] [patch] doublefree and misalign in dec_audio.c
Luca Barbato
lu_zero at gentoo.org
Mon Feb 13 19:12:10 CET 2006
Reimar Döffinger wrote:
> Hi,
> On Mon, Feb 13, 2006 at 12:45:01PM +0100, Luca Barbato wrote:
>> 1 the af got freed on af_uninit so the other free is wrong.
>
> But _that_ free is for the af_stream_t *, AFAICT af_uninit only frees
> the af_instance_t * 's
You are right, was too tired yesterday night, I'm rechecking if that
issue was just due the misalignment fixed before
>
>> 2 the buffers aren't aligned, so some altivec/SIMD code may not behave
>> correctly at random if is expecting aligned space.
>
> Yes, that is fine with me, I'd say it should be applied.
Good =)
lu
--
Luca Barbato
Gentoo/linux Developer Gentoo/PPC Operational Leader
http://dev.gentoo.org/~lu_zero
More information about the MPlayer-dev-eng
mailing list