summaryrefslogtreecommitdiff
path: root/ospfd/ospf_errors.c
diff options
context:
space:
mode:
authorNitin Soni <nsoni@cumulusnetworks.com>2019-01-24 00:44:42 -0800
committerNitin Soni <nsoni@cumulusnetworks.com>2019-01-27 20:27:58 -0800
commit9b18d58e17d7e84f17114e5f4a6461dd7e250580 (patch)
tree8fc85feba66b6c0c5f1997cc0dd13b039cd06f52 /ospfd/ospf_errors.c
parent262d4dda2aa009f200370a4c4635899ceb85ab90 (diff)
ospfd: ospfd core if hello packet exceeds link MTU
Ospfd cored because of an assert when we try to write more than the MTU size to the ospf packet buffer stream. The problem is - we allocate only MTU sized buffer. The expectation is that Hello packets are never large enough to approach MTU. Instead of crashing, this fix discards hello and logs an error. One should not have so many neighbors behind an interface. Ticket: CM-22380 Signed-off-by: Nitin Soni <nsoni@cumulusnetworks.com> Reviewed-by: CCR-8204
Diffstat (limited to 'ospfd/ospf_errors.c')
-rw-r--r--ospfd/ospf_errors.c9
1 files changed, 9 insertions, 0 deletions
diff --git a/ospfd/ospf_errors.c b/ospfd/ospf_errors.c
index 566fc29202..b912a80692 100644
--- a/ospfd/ospf_errors.c
+++ b/ospfd/ospf_errors.c
@@ -170,6 +170,15 @@ static struct log_ref ferr_ospf_err[] = {
.suggestion = "Gather log files and open an issue",
},
{
+ .code = EC_OSPF_LARGE_HELLO,
+ .title = "OSPF Encountered a Large Hello",
+ .description = "OSPF attempted to send a Hello larger than MTU "
+ "but did not",
+ .suggestion = "Too many neighbors configured on a single interface."
+ " Suggestion is to decrease the number of neighbors on"
+ " a single interface/subnet"
+ },
+ {
.code = END_FERR,
}
};