diff options
author | Quentin Young <qlyoung@cumulusnetworks.com> | 2018-05-29 10:17:08 +0200 |
---|---|---|
committer | Quentin Young <qlyoung@cumulusnetworks.com> | 2018-05-29 21:06:46 +0200 |
commit | c2ca5ee66d8f11672be87c079456158b9860e99d (patch) | |
tree | acf3a06c5c54a0c6125711fe9238bde3ac0a8d00 /zebra/zserv.h | |
parent | zebra: rebase zserv-mt (diff) | |
download | frr-c2ca5ee66d8f11672be87c079456158b9860e99d.tar.xz frr-c2ca5ee66d8f11672be87c079456158b9860e99d.zip |
zebra: fix race condition in i/o pthread shutdown
I mistakenly used an external mechanism to cause a pthread to shut
itself down instead of using the one built into frr_pthread.[ch]. This
created a race condition whereby a pthread could schedule work onto a
dead pthread and cause it to reanimate.
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
Diffstat (limited to 'zebra/zserv.h')
-rw-r--r-- | zebra/zserv.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/zebra/zserv.h b/zebra/zserv.h index 2a681552d..78cc200fa 100644 --- a/zebra/zserv.h +++ b/zebra/zserv.h @@ -53,9 +53,6 @@ struct zserv { /* Client pthread */ struct frr_pthread *pthread; - /* Whether the thread is waiting to be killed */ - _Atomic bool dead; - /* Client file descriptor. */ int sock; |