summaryrefslogtreecommitdiffstats
path: root/eigrpd/eigrp_fsm.c (follow)
Commit message (Collapse)AuthorAgeFilesLines
* *: generously apply constDavid Lamparter2019-12-021-1/+1
| | | | | | const const const your boat, merrily down the stream... Signed-off-by: David Lamparter <equinox@diac24.net>
* eigrpd: Convert eigrp_neighor.c to not use VRF_DEFAULTDonald Sharp2019-08-071-3/+3
| | | | Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Remove VRF_DEFAULT from eigrp_fsm.cDonald Sharp2019-08-071-3/+1
| | | | Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Update eigrp_topology.c to not use VRF_DEFAULTDonald Sharp2019-08-071-8/+10
| | | | | | | Push out the pass in of struct eigrp to all functions in eigrp_topology.c so we do not lookup the eigrp pointer. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Add `router eigrp AS [vrf NAME]` and various stuffDonald Sharp2019-08-071-1/+1
| | | | | | | Add the ability to parse `router eigrp AS [vrf NAME]` and modify eigrp_lookup to actually handle a vrf_id for us. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* *: remove trailing newlines from zlog messagesQuentin Young2019-03-141-4/+4
| | | | | | Zlog puts its own newlines on, and doing this makes logs look nasty. Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
* *: remove unnecessary semicolon from switchesQuentin Young2019-02-261-1/+1
| | | | Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
* *: spelchekDavid Lamparter2018-10-251-2/+2
| | | | Signed-off-by: David Lamparter <equinox@diac24.net>
* *: list_delete_and_null() -> list_delete()David Lamparter2018-10-021-7/+7
| | | | Signed-off-by: David Lamparter <equinox@diac24.net>
* *: fix config.h/zebra.h include orderDavid Lamparter2018-09-081-1/+1
| | | | | | | | config.h (or, transitively, zebra.h) must be the first include file listed for autoconf things like _GNU_SOURCE and _POSIX_C_SOURCE to work correctly. Signed-off-by: David Lamparter <equinox@diac24.net>
* eigrpd: null check (Coverity 1455334 and 4 alike)paco2018-06-211-3/+5
| | | | | | | | Coverity issues fixed with this commit: 1455334 1455335 1455336 1455337 1455339 Signed-off-by: F. Aragon <paco@voltanet.io>
* *: use C99 standard fixed-width integer typesQuentin Young2018-03-271-2/+2
| | | | | | | | | | | | | | | | | | | | | | The following types are nonstandard: - u_char - u_short - u_int - u_long - u_int8_t - u_int16_t - u_int32_t Replace them with the C99 standard types: - uint8_t - unsigned short - unsigned int - unsigned long - uint8_t - uint16_t - uint32_t Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
* *: conform with COMMUNITY.md formatting rules, via 'make indent'Lou Berger2018-03-061-23/+15
| | | | Signed-off-by: Lou Berger <lberger@labn.net>
* eigrpd: Fix an issue found with metric changeDonald Sharp2017-10-291-4/+24
| | | | | | | | | | | A past commit modified the change value to an enum but did not bother to fix all the places where change was used. Fix this. Additionally add some more output to the fsm prefix string about the change. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Add ability to show packet type in logDonald Sharp2017-10-291-2/+28
| | | | | | | Allow us to examine the packet type that caused us to change state. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Create enum for states and string name for displayDonald Sharp2017-10-291-4/+30
| | | | | | | Create an enum for the different states and create a string name display handler. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Create a function to return a string of prefix stateDonald Sharp2017-10-291-3/+21
| | | | Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* *: Convert list_delete(struct list *) to ** to allow nullingDonald Sharp2017-10-051-7/+7
| | | | | | | | | | | | | | | | | | | Convert the list_delete(struct list *) function to use struct list **. This is to allow the list pointer to be nulled. I keep running into uses of this list_delete function where we forget to set the returned pointer to NULL and attempt to use it and then experience a crash, usually after the developer has long since left the building. Let's make the api explicit in it setting the list pointer to null. Cynical Prediction: This code will expose a attempt to use the NULL'ed list pointer in some obscure bit of code. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: eigrp_neighbor_entry to eigrp_nexthop_entryDonald Sharp2017-08-271-19/+19
| | | | | | | | The struct 'eigrp_neighbor_entry' really represents a nexthop for a prefix. Rename the structure to better represent what we are representing. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Refactor FSM callingDonald Sharp2017-08-241-2/+3
| | | | Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Cleanup list accessorsDonald Sharp2017-08-241-54/+38
| | | | Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Remove extra newline in debugDonald Sharp2017-08-171-1/+1
| | | | Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Cleanup eigrp_get_fsm_eventDonald Sharp2017-08-161-19/+20
| | | | | | Make this function a bit more legible Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Make eigrp_get_fsm_event use metric changedDonald Sharp2017-08-161-18/+10
| | | | | | | Refactor eigrp_get_fsm_event a tiny bit to let all the states get the new metric. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* *: reindentreindent-master-afterwhitespace / reindent2017-07-171-417/+410
| | | | | | indent.py `git ls-files | pcregrep '\.[ch]$' | pcregrep -v '^(ldpd|babeld|nhrpd)/'` Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* *: do not take address of packed memberQuentin Young2017-05-261-2/+2
| | | | | | May result in alignment errors on certain platforms Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
* *: make consistent & update GPLv2 file headersDavid Lamparter2017-05-151-6/+3
| | | | | | | | | | | The FSF's address changed, and we had a mixture of comment styles for the GPL file header. (The style with * at the beginning won out with 580 to 141 in existing files.) Note: I've intentionally left intact other "variations" of the copyright header, e.g. whether it says "Zebra", "Quagga", "FRR", or nothing. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* eigrpd: Cleanup tab/spacing of the *.c filesDonald Sharp2017-04-081-385/+439
| | | | Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrpd: Fix massive memory leakDonald Sharp2017-04-061-25/+58
| | | | | | | | | | | | | Each call into eigrp_topology_get_successors would leak the list created. As routes worked their way through the FSM we would leak memory left and right. Modify the eigrp_topology_get_successor to return NULL when there are no SUCCESORS. Clean up some dead code. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
* eigrp: Initial CommitDonald Sharp2017-03-091-0/+502
Please Note, I will be redoing this commit message with more information. Additionally I will rework the lib/* changes into their own commits. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>