Re: [patch] Defining apt sources using codename instead of suite
On Sun, Oct 23, 2005 at 06:29:48PM +0200, Frans Pop wrote:
> On Sunday 23 October 2005 18:11, Jens Seidel wrote:
> > s/de/the/
>
> Already noticed by svenl on IRC :-)
OK.
> > Isn't free(directory); ... missing here?
>
> No, I don't think so.
>
> > > + free(hostname);
> > > + free(directory);
> ^^^^^^^^^^^^^^^^
> > > + free(suite);
No, it is missing! You leave the function via explicit return without
freeing your allocated memory first!
But you mentioned that the condition in which you call return should not
happen, so ...
Jens
Reply to: