On Mon, Jan 12, 2015 at 10:02 AM, Todd C. Miller
<[email protected]> wrote:
> On Mon, 12 Jan 2015 09:12:02 -0600, Brent Cook wrote:
>
>>  - Nothing seems to free the result of host_dns(), so add
>>    host_dns_free() and call after each query.
>>  - If imsg_add fails, it frees buf. Avoid dereferencing the freed buf
>>    afterward in imsg_close().
>
> That looks good to me.
>
>  - todd

Unfortunately, this caused a regression:

 1. bring down networking
 2. start ntpd
 3. start networking

Result is, it doesn't properly resolve the hosts on the next poll.

I just got finished bisecting to point to this as the problem diff.
Will be back online in about 30 minutes when I'm near a power adapter
again.

Reply via email to