summaryrefslogtreecommitdiffstats
path: root/runtime/wtp.c
diff options
context:
space:
mode:
authorRainer Gerhards <rgerhards@adiscon.com>2009-04-02 14:41:32 +0200
committerRainer Gerhards <rgerhards@adiscon.com>2009-04-02 14:41:32 +0200
commit01adeab0cba21ad6193addf1a4e90689b507d092 (patch)
tree3fd605896c1a18f73b877ecb578a5a62f15e6723 /runtime/wtp.c
parent589f8af56cac4a4b55de757834b6e632a199e395 (diff)
downloadrsyslog-01adeab0cba21ad6193addf1a4e90689b507d092.tar.gz
rsyslog-01adeab0cba21ad6193addf1a4e90689b507d092.tar.xz
rsyslog-01adeab0cba21ad6193addf1a4e90689b507d092.zip
preparing for 3.20.5 releasev3.20.5
Diffstat (limited to 'runtime/wtp.c')
-rw-r--r--runtime/wtp.c8
1 files changed, 8 insertions, 0 deletions
diff --git a/runtime/wtp.c b/runtime/wtp.c
index fcefa1d8..3e3ff09a 100644
--- a/runtime/wtp.c
+++ b/runtime/wtp.c
@@ -196,6 +196,14 @@ wtpProcessThrdChanges(wtp_t *pThis)
FINALIZE;
}
+ /* Note: there is a left-over potential race condition below:
+ * pThis->bThrdStateChanged may be re-set by another thread while
+ * we work on it and thus the loop may terminate too early. However,
+ * there are no really bad effects from that so I perfer - for this
+ * version - to live with the problem as is. Not a good idea to
+ * introduce that large change into the stable branch without very
+ * good reason. -- rgerhards, 2009-04-02
+ */
do {
/* reset the change marker */
pThis->bThrdStateChanged = 0;