Marius Nünnerich writes: > On Sun, Feb 20, 2011 at 21:34, Ashish SHUKLA <ash...@freebsd.org> wrote: >> Hi Marius, >> >> Marius Nünnerich writes: >> >> [...] >> >>> Ok, I applied the patch and reinstalled the port. I will leave it >>> running for a few days and report back. So far it's looking good. >>> Thank you very much! :) >>
> OK, I had it running for a while now with your fix. It's not leaking anymore. > There is still one close() missing before a return in that same function. Yes, you're right. I've updated the original diff[1] in case someone like to try it. I'll commit it in a day or two. >> I talked to Paul (the upstream author of pdnsd), and he suggested running >> dns/pdnsd with following command-line (requires devel/valgrind port): >> >> % valgrind --tool=memcheck --leak-check=yes --track-fds==yes pdnsd --nodaemon > I'll try this too if I can find the time, but so far it doesn't seem > to leak memory. Cool. References: [1] http://people.freebsd.org/~ashish/diffs/pdnsd-sockets-leak.diff Thanks -- Ashish SHUKLA | GPG: F682 CDCC 39DC 0FEA E116 20B6 C746 CFA9 E74F A4B0 freebsd.org!ashish | http://people.freebsd.org/~ashish/ Avoid Success At All Costs !!
pgpMAxlw4FcuQ.pgp
Description: PGP signature