BUG/MINOR: spoe: Don't release the context buffer in .check_timeouts callbaclk

It is better to let spoe_stop_processing release this buffer because, in
.check_timeouts callback, we lack information to know if it should be release or
not. For instance, if the processing timeout is reached while the SPOE applet
receives the reply, it is preferable to ignore the timeout and process the
result.

This patch should be backported in 1.8.
This commit is contained in:
Christopher Faulet 2018-03-20 16:09:20 +01:00 committed by Willy Tarreau
parent 84c844eb12
commit ac580608d7

View File

@ -2969,10 +2969,8 @@ spoe_check_timeouts(struct stream *s, struct filter *filter)
{
struct spoe_context *ctx = filter->ctx;
if (tick_is_expired(ctx->process_exp, now_ms)) {
if (tick_is_expired(ctx->process_exp, now_ms))
s->pending_events |= TASK_WOKEN_MSG;
spoe_release_buffer(&ctx->buffer, &ctx->buffer_wait);
}
}
/* Called when we are ready to filter data on a channel */