diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2019-09-24 14:24:10 +0200 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2019-09-24 14:29:46 +0200 |
| commit | a3116965bb2332901c5645ef119087dbd148a95f (patch) | |
| tree | b6c88a146d95c6a3c034750a3e61caecc34a6681 /bgpd/bgpd.h | |
| parent | pbrd: We should be looking for v4 and v6 nexthop types (diff) | |
| download | frr-a3116965bb2332901c5645ef119087dbd148a95f.tar.xz frr-a3116965bb2332901c5645ef119087dbd148a95f.zip | |
bgpd: rmap_type is 8 bit but we have 9 bits of flags
The newly added PEER_RMAP_TYPE_AGGREGATE flag is setup to
be the 9th bit:
But the flag we are putting it into:
uint8_t rmap_type;
is 8 bits. Adjust the size.
Found by Coverity SA Scan
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to '')
| -rw-r--r-- | bgpd/bgpd.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bgpd/bgpd.h b/bgpd/bgpd.h index 9d45d9698..320f1ec96 100644 --- a/bgpd/bgpd.h +++ b/bgpd/bgpd.h @@ -1202,7 +1202,7 @@ struct peer { uint8_t last_reset_cause[BGP_MAX_PACKET_SIZE]; /* The kind of route-map Flags.*/ - uint8_t rmap_type; + uint16_t rmap_type; #define PEER_RMAP_TYPE_IN (1 << 0) /* neighbor route-map in */ #define PEER_RMAP_TYPE_OUT (1 << 1) /* neighbor route-map out */ #define PEER_RMAP_TYPE_NETWORK (1 << 2) /* network route-map */ |