This helps with debugging dns and reentrancy problems.
Also, assert in ensure_resolving that we have an address. This makes
it slightly clearer that callers are expected to have checked this.
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---
New patch in v2 of the series.
ca_buf.sin.sin_port=htons(st->remoteport);
ca_buf.sin.sin_addr=*address;
ca_use=&ca_buf;
+ slog(st,LOG_STATE,"resolution of %s completed: %s",
+ st->address, comm_addr_to_string(ca_use));;
} else {
slog(st,LOG_ERROR,"resolution of %s failed",st->address);
ca_use=0;
if (st->resolving)
return True;
+ assert(st->address);
+
/* resolver->request might reentrantly call site_resolve_callback
* which will clear st->resolving, so we need to set it beforehand
* rather than afterwards; also, it might return False, in which