[FFmpeg-devel] [PATCH] tcp.c/udp.c memleak?
Sat Aug 23 00:03:07 CEST 2008
On Fri, Aug 22, 2008 at 05:26:45PM -0400, Ronald S. Bultje wrote:
> Hi Michael,
> On Thu, Aug 21, 2008 at 1:02 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Wed, Aug 20, 2008 at 11:09:27PM -0400, Ronald S. Bultje wrote:
> >> Hi Michael,
> >> On Wed, Aug 20, 2008 at 10:42 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> >> > On Wed, Aug 20, 2008 at 02:03:12PM -0400, Ronald S. Bultje wrote:
> >> >> Hi,
> >> >>
> >> >> tcpmemleak.patch: I think the malloc should go after the init &
> >> >> paramchecks otherwise the return in ff_network_init() could be a
> >> >> memleak. In tcp.c, I move the malloc down,
> >> >
> >> > i think it should be moved over all goto fail* which would allow
> >> > the fail case to be simplified
> >> Hm, ok, like $attached?
> What about the first patch?
that mixes unrelated changes as well
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel