diff options
| author | Philippe Guibert <philippe.guibert@6wind.com> | 2024-10-28 18:20:13 +0100 |
|---|---|---|
| committer | Philippe Guibert <philippe.guibert@6wind.com> | 2024-12-02 16:54:04 +0100 |
| commit | 2dc693e808985ed7e15a12dd5a7bb8f71a60a9d3 (patch) | |
| tree | 038e51e69fd8950605ae55138bc4081fa5fca4cb /lib/frr_pthread.c | |
| parent | 0af653eaf7861219284f82b97b9c4a37dc568985 (diff) | |
topotests: fix bmp_collector handling of empty as-path
When configuring the bgp_bmp test with an additional
peer that sends empty AS-PATH, the bmp collector is stopping:
> [2024-10-28 17:41:51] Finished dissecting data from ('192.0.2.1', 33922)
> [2024-10-28 17:41:52] Data received from ('192.0.2.1', 33922): length 195
> [2024-10-28 17:41:52] Got message type: <class 'bmp.BMPRouteMonitoring'>
> [2024-10-28 17:41:52] unpack_from requires a buffer of at least 2 bytes for unpacking 2 bytes at offset 0 (actual buffer size is 0)
> [2024-10-28 17:41:52] TCP session closed with ('192.0.2.1', 33922)
> [2024-10-28 17:41:52] Server shutting down on 192.0.2.10:1789
The parser attempts to read an empty AS-path and considers the length
value as a length in bytes, whereas RFC mentions this value as
definining the number of AS-PAths. Fix this in the parser.
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Diffstat (limited to 'lib/frr_pthread.c')
0 files changed, 0 insertions, 0 deletions