BUG/MEDIUM: counters: fix stick-table entry leak when using track-sc2 in connection

In 1.5-dev19, commit e25c917 ("MEDIUM: counters: add support for tracking
a third counter") introduced the third track counter. However, there was
a hard-coded test in the accept() error path to release only sc0 and sc1.
So it seems that if tracking sc2 at the connection level and deciding to
reject once the track-sc2 has been done, there could be some leaking of
stick-table entries which remain marked used forever, thus which can never
be purged nor expired. There's no memory leak though, it's just that
entries are unexpirable forever.

The simple solution consists in removing the test and always calling
the inline function which iterates over all entries.

diff --git a/src/session.c b/src/session.c
index 8241d069988de0723b8807875fc6e510e021ec47..600d536359e47244420e645bf453a280c9ad6e35 100644 (file)
--- a/src/session.c
+++ b/src/session.c
@@ -230,8 +230,7 @@ int session_accept(struct listener *l, int cfd, struct sockaddr_storage *addr)
        task_free(t);
  out_free_session:
        p->feconn--;
-       if (s->stkctr[0].entry || s->stkctr[1].entry)
-               session_store_counters(s);
+       session_store_counters(s);
        pool_free2(pool2_session, s);
  out_free_conn:
        cli_conn->flags &= ~CO_FL_XPRT_TRACKED;