[pacman-dev] small memleak patch

VMiklos vmiklos at frugalware.org
Fri Dec 23 06:52:01 EST 2005


On Thu, Dec 22, 2005 at 05:22:32PM +0100, Essien Ita Essien <essiene at datavibe.net> wrote:
> +	db_close(handle->db_local); /*verify no problems, like double free (basically, is this the right way to handle pmdb_t free/close?*/

yes, this is the right way, but this is done in alpm.c, alpm_release()
there will be no double free as there is already a check there for
preventing this

so this one is unnecessary (the others looks good to me)

the format (cvs diff -u) is ok, too :)

udv / greetings,
VMiklos

-- 
Developer of Frugalware Linux, to make things frugal - http://frugalware.org




More information about the pacman-dev mailing list