Hi guys,
On Wed, Sep 02, 2015 at 08:44:21AM +0200, Cyril Bonté wrote:
> I haven't made some tests yet, but I guess the issue is on srv_conn due
> to this part of code in proto_http.c :
> if (((s->txn->flags & TX_CON_WANT_MSK) != TX_CON_WANT_KAL) ||
> !si_conn_ready(&s->si[1])) {
> si_release_endpoint(&s->si[1]);
> srv_conn = NULL;
> }
>
> [...]
>
> if (prev_status == 401 || prev_status == 407) {
> [...]
> s->txn->flags |= TX_PREFER_LAST;
> srv_conn->flags |= CO_FL_PRIVATE;
> }
>
> In some conditions, srv_conn is set to NULL but is then used later.
Good catch, thanks for this! I've merged the fix. Fortunately it
only impacts 1.6 since the connection reuse code.
Thanks,
Willy
