On 19 Dec 2001, Dave Dykstra <dwd@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