diff options
| author | Quentin Young <qlyoung@cumulusnetworks.com> | 2020-01-03 21:18:49 -0500 |
|---|---|---|
| committer | Quentin Young <qlyoung@cumulusnetworks.com> | 2020-01-03 22:33:52 -0500 |
| commit | ab12ca856cc7acfe7ccaa0849919181f53a3dc32 (patch) | |
| tree | 3838795277313d17741979de25211494dc8f4de6 /doc/manpages/conf.py | |
| parent | b833cc1ff473e5de43a6c33b520024c943db180c (diff) | |
zebra: fix multiple bfd buffer issues
Whatever this BFD re-transmission function is had a few problems.
1. Used memcpy instead of the (more concise) stream APIs, which include
bounds checking.
2. Did not sufficiently check packet sizes.
Actually, 2) is mitigated but is still a problem, because the BFD header
is 2 bytes larger than the "normal" ZAPI header, while the overall
message size remains the same. So if the source message being duplicated
is actually right up against the ZAPI_MAX_PACKET_SIZ, you still can't
fit the whole message into your duplicated message. I have no idea what
the intent was here but at least there's a warning if it happens now.
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
Diffstat (limited to 'doc/manpages/conf.py')
0 files changed, 0 insertions, 0 deletions
