Technically this is a leak, but it does no harm to leave them there:
- It may aid in debugging
- future features (try pronouncing that! ;-) may use these values for
other purposes
- additionally, while the pointers are freed, they're not set to NULL,
which makes them unsafe dirty pointers. That's actually worse than not
being freed, in my opinion.
setup_servers(servers, genconf.modernaddr, genconf.modernport);
dousers(genconf.user, genconf.group);
- g_free(genconf.user);
- g_free(genconf.group);
- g_free(genconf.modernaddr);
- g_free(genconf.modernport);
-
serveloop(servers);
}