MEDIUM: frontend: make ->accept only return +/-1
This function was specified as being able to return 3 states, which had
repercussions to the stream accept function. It was used at the time
when the frontend would do the monitoring itself. This is not the case
anymore, so let's simplify this.
diff --git a/src/stream.c b/src/stream.c
index 256b283..1bcd2eb 100644
--- a/src/stream.c
+++ b/src/stream.c
@@ -213,16 +213,8 @@
else if (appctx)
s->si[0].flags |= SI_FL_WAIT_DATA;
- /* FIXME: we shouldn't restrict ourselves to connections but for now
- * the only ->accept() only works with sessions.
- */
- if (conn && p->accept && (ret = p->accept(s)) <= 0) {
- /* Either we had an unrecoverable error (<0) or work is
- * finished (=0, eg: monitoring), in both situations,
- * we can release everything and close.
- */
- goto out_free_strm;
- }
+ if (conn && p->accept && p->accept(s) < 0)
+ goto out_fail_accept;
if (conn) {
/* if logs require transport layer information, note it on the connection */
@@ -241,7 +233,7 @@
return 1;
/* Error unrolling */
- out_free_strm:
+ out_fail_accept:
LIST_DEL(&s->list);
pool_free2(pool2_stream, s);
out_return: