[pacman-dev] small memleak patch

Essien Ita Essien essiene at datavibe.net
Thu Dec 29 08:36:55 EST 2005



VMiklos wrote:
> 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)

oh cool.

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

bingo!!! :)
> 
> udv / greetings,
> VMiklos
> 

btw, sorry for replying this late... i had to travel, and just got 
connected back to the net today... so back to hacking ;)






More information about the pacman-dev mailing list