tree b341a95e4bfc497992e33a6aab83ee36e72af777
parent 7b5810dec723cced1ecff82ca3ddd4de0f9a5e28
author Willy Tarreau <w@1wt.eu> 1588688016 +0200
committer Christopher Faulet <cfaulet@haproxy.com> 1590387742 +0200

BUG/MINOR: threads: fix multiple use of argument inside HA_ATOMIC_UPDATE_{MIN,MAX}()

Just like in previous patch, it happens that HA_ATOMIC_UPDATE_MIN() and
HA_ATOMIC_UPDATE_MAX() would evaluate the (val) argument up to 3 times.
However this time it affects both thread and non-thread versions. It's
strange because the copy was properly performed for the (new) argument
in order to avoid this. Anyway it was done for the "val" one as well.

A quick code inspection showed that this currently has no effect as
these macros are fairly limited in usage.

It would be best to backport this for long-term stability (till 1.8)
but it will not fix an existing bug.

(cherry picked from commit a4d9ee3d1cc7b321be420742263b67620036bb71)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 1b04543c708364ab403ecd09758eafb23b6de916)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>