[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <AE90C24D6B3A694183C094C60CF0A2F6026B7181@saturn3.aculab.com>
Date: Thu, 7 Mar 2013 10:06:41 -0000
From: "David Laight" <David.Laight@...LAB.COM>
To: "jb" <jb.1234abcd@...il.com>, <netdev@...r.kernel.org>
Subject: RE: [PATCH] iputils: fix memory leaks
> Fix memory leaks
...
> +#ifdef USE_IDN
> + free(idn);
> + idn = NULL;
> +#endif
> exit(2);
There is no point calling free() before exit(), nor any need
to free items that aren't allocated inside loops.
David
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists