BUG/MAJOR: stick-table: don't process store-response rules for applets

The commit bc7c207f74 ("BUG/MAJOR: stick-tables: do not try to index a
server name for applets") tried to catch applets case when we tried to index
the server name. However, there is still an issue. The applets are
unconditionally casted to servers and this bug exists since a while. it's
just luck if it doesn't crash.

Now, when store rules are processed, we skip the rule if the stream's target
is not a server or, of course, if it is a server but the "non-stick" option
is set. However, we still take care to release the sticky session.

This patch must be backported to all stable versions.

diff --git a/src/stream.c b/src/stream.c
index b19519f84adee35cfa509039f54d6ef806b3307e..f45f535110521a0a75ec578f95840bf6e60207cc 100644 (file)
--- a/src/stream.c
+++ b/src/stream.c
@@ -1411,7 +1411,7 @@ static int process_store_rules(struct stream *s, struct channel *rep, int an_bit
                struct dict_entry *de;
                struct stktable *t = s->store[i].table;
 
-               if (objt_server(s->target) && __objt_server(s->target)->flags & SRV_F_NON_STICK) {
+               if (!objt_server(s->target) || (__objt_server(s->target)->flags & SRV_F_NON_STICK)) {
                        stksess_free(s->store[i].table, s->store[i].ts);
                        s->store[i].ts = NULL;
                        continue;