[FFmpeg-devel] One memory leakage in AAC/AC3 parser in ffmpeg

Michael Niedermayer michaelni
Thu Jun 19 10:38:40 CEST 2008


On Thu, Jun 19, 2008 at 09:34:00AM +0200, David LIU wrote:
> On Wed, Jun 18, 2008 at 11:58:02AM +0200, David LIU wrote:
> >> Hi,
> >> 
> >> Please review patch and let me know reject or accept
> >> 
> >> Best regards
> 
> [...]
> 
> >> (Also avoid top-posting.)
>    ^^^^^^^^^^^^^^^^^^^^^^^
> 
> >besides that, ff_parse_close() should be used not a new redundant function
> added.
> 
> [David]: There is ParseContext type in ff_parse_close(). I have to use
> AACAC3ParseContext type and that's why I create new ff_parse_close2().

AACAC3ParseContext contains a ParseContext, move it to the top and
AACAC3ParseContext is a ParseContext

[..]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080619/3c3ef497/attachment.pgp>



More information about the ffmpeg-devel mailing list