[Frugalware-devel] Re: setup repo

VMiklos vmiklos at frugalware.org
Tue Mar 27 21:18:34 CEST 2007


second private mail ;) (oops, replied privately first time)

Na Tue, Mar 27, 2007 at 02:45:40PM -0300, Cláudio Henrique <claudiohfg at gmail.com> pisal(a):
> still in util.c, why not use

> system(g_strdup_printf("rm -rf %s", path));

> instead of

we have that function in util.c, so feel free to send a patch that
changes this, you have reason. not speaking about the above line causes
a memleak, the proper code would be:

char *ptr = g_strdup_printf("command %s", path);
system(ptr);
free(ptr);

if you find other similar problems elsewhere (which i likely), then
please send a patch to correct those problems

thanks,
VMiklos

-- 
developer of Frugalware Linux - http://frugalware.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : /pipermail/attachments/20070327/02251ab3/attachment.bin


More information about the Frugalware-devel mailing list