missing call to freeaddrinfo() in socket.c?

Martin Pool mbp at samba.org
Sat Mar 16 19:56:12 EST 2002


On 19 Dec 2001, Dave Dykstra <dwd at bell-labs.com> wrote:
> Martin,
> 
> I just read the Linux getaddrinfo() man page and I see that each
> getaddrinfo() should be followed by a freeaddrinfo().  Neglecting the cases
> that result in errors which I presume end up causing rsync to exit, it
> looks to me like the call now in the try_bind_local() function (in the
> current rsync CVS) does not have a corresponding freeaddrinfo().  The
> previous version of the code didn't have it either, so it's not that it got
> dropped in the recent re-arrangement.

Yes, I think it was missing from the kame patch.  Fixed in CVS,
thanks.

-- 
Martin 




More information about the rsync mailing list