summaryrefslogtreecommitdiffstats
path: root/bfdd/control.c (follow)
Commit message (Collapse)AuthorAgeFilesLines
* bfdd: Do not explicitly set the thread pointer to NULLDonatas Abraitis2021-10-041-1/+0
| | | | | | | | FRR should only ever use the appropriate THREAD_ON/THREAD_OFF semantics. This is espacially true for the functions we end up calling the thread for. Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* * : update signature of thread_cancel apiMark Stapp2020-10-231-17/+4
| | | | | | | | Change thread_cancel to take a ** to an event, NULL-check before dereferencing, and NULL the caller's pointer. Update many callers to use the new signature. Signed-off-by: Mark Stapp <mjs@voltanet.io>
* bfdd: simplify code flowRafael Zalamena2020-04-141-6/+2
| | | | | | | Don't attempt to handle out-of-memory situations: XMALLOC/XCALLOC will `assert` if there is no memory left. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* bfdd: use standard log macrosRafael Zalamena2020-04-101-23/+23
| | | | | | Remove old log_* macros and standardize on FRR's logging infrastructure. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* *: don't null after XFREE; XFREE does this itselfQuentin Young2020-02-031-1/+0
| | | | Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
* bfdd: fix format stringsDavid Lamparter2020-01-211-1/+1
| | | | | | ... now that we get warnings about them ... Signed-off-by: David Lamparter <equinox@diac24.net>
* bfdd: Handling local and remote admin-downSumitAgarwal1232019-10-291-2/+2
| | | | | | | | | | | | | | | | | | | | Scenarios where this code change is required: 1. BFD is un-configured from BGP at remote end. Neighbour BFD sends ADMIN_DOWN state, but BFD on local side will send DOWN to BGP, resulting in BGP session DOWN. Removing BFD session administratively shouldn't bring DOWN BGP session at local or remote. 2. BFD is un-configured from BGP or shutdown locally. BFD will send state DOWN to BGP resulting in BGP session DOWN. (This is akin to saying do not use BFD for BGP) Removing BFD session administratively shouldn't bring DOWN BGP session at local or remote. Signed-off-by: Sayed Mohd Saquib sayed.saquib@broadcom.com
* *: do not check XMALLOC / XCALLOC for null retQuentin Young2019-02-261-25/+0
| | | | | | They never return NULL Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
* *: Rename backet to bucketTim Bray2019-02-251-2/+2
| | | | | | Presume typo from original author Signed-off-by: Tim Bray <tim@kooky.org>
* bfdd: fix coverity scan issue (CID 1472622)Rafael Zalamena2018-08-161-3/+3
| | | | | | | Don't use the stack variable, but what we have recorded in our buffered data on the heap. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* bfdd: clean up header inclusionRafael Zalamena2018-08-081-9/+0
| | | | | | | Most of the headers we need are included by zebra.h, so lets simplify this. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* bfdd: re-route PTM-BFD daemon messagesRafael Zalamena2018-08-081-0/+3
| | | | | | | | | | | When `bfdd` is enabled - which it is by default - re-route the PTM-BFD messages to the FRR's internal BFD daemon instead of the external PTM daemon. This will help the migration of BFD implementations and avoid duplicating code. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
* bfdd: imported new daemon source codeRafael Zalamena2018-08-081-0/+901
Import source code from external `bfdd` daemon ported from Cumulus PTM. Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>