BUG/MINOR: http/threads: atomically increment the error snapshot ID

Let's use an atomic increment for the error snapshot, as we'd rather
not assign the same ID to two errors happening in parallel. It's very
unlikely that it will ever happen though.

This patch must be backported to 1.8 with the other one it relies on
("MINOR: thread: implement HA_ATOMIC_XADD()").
This commit is contained in:
Willy Tarreau 2018-09-07 11:29:59 +02:00
parent 044fd5bc2c
commit e9e878a056

View File

@ -8053,7 +8053,7 @@ void http_capture_bad_message(struct proxy *proxy, struct error_snapshot *es, st
memset(&es->src, 0, sizeof(es->src)); memset(&es->src, 0, sizeof(es->src));
es->state = state; es->state = state;
es->ev_id = error_snapshot_id++; es->ev_id = HA_ATOMIC_XADD(&error_snapshot_id, 1);
es->b_flags = chn->flags; es->b_flags = chn->flags;
es->s_flags = s->flags; es->s_flags = s->flags;
es->t_flags = s->txn->flags; es->t_flags = s->txn->flags;