On Sat, May 30, 2015 at 01:40:24PM +0200, Claudio Jeker wrote: > New version that now passes all regress tests. It got a fair bit more > complex because the F_TLSINSPECT case results in multiple entries of > relay_connect and relay_connected. So I switched to use a state variable > instead of a flag. It seems to work (and also works in the EMFILE/ENFILE > case). At least it "worked" for me by forcing that code path all the time.
> @@ -1673,6 +1697,7 @@ relay_close(struct rsession *con, const > event_add(&rlay->rl_ev, NULL); > } > } > + con->se_out.state = INIT; > > if (con->se_out.buf != NULL) > free(con->se_out.buf); Resetting the state is not neccessary, free(con) will invalidate the storage. anyway, OK bluhm@
