diff options
| author | Rainer Gerhards <rgerhards@adiscon.com> | 2009-04-02 14:41:32 +0200 |
|---|---|---|
| committer | Rainer Gerhards <rgerhards@adiscon.com> | 2009-04-02 14:41:32 +0200 |
| commit | 01adeab0cba21ad6193addf1a4e90689b507d092 (patch) | |
| tree | 3fd605896c1a18f73b877ecb578a5a62f15e6723 /runtime/wtp.c | |
| parent | 589f8af56cac4a4b55de757834b6e632a199e395 (diff) | |
| download | rsyslog-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.c | 8 |
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; |