diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2019-10-16 16:25:19 +0200 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2019-10-16 19:38:29 +0200 |
| commit | 05864da7913a4874ac3e0c808930114e3549b265 (patch) | |
| tree | cca27d22eb47277a512055cc8c2e26f9134bfbb5 /bgpd/bgp_flowspec_vty.c | |
| parent | doc: Add MRAI timer documentation to bgp docs (diff) | |
| download | frr-05864da7913a4874ac3e0c808930114e3549b265.tar.xz frr-05864da7913a4874ac3e0c808930114e3549b265.zip | |
bgpd: `struct bgp_path_info *`->attr must not be NULL
We make the assumption that ->attr is not NULL throughout
the code base. We are totally inconsistent about application
of this though.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'bgpd/bgp_flowspec_vty.c')
| -rw-r--r-- | bgpd/bgp_flowspec_vty.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bgpd/bgp_flowspec_vty.c b/bgpd/bgp_flowspec_vty.c index 2d6523ed3..80384c12c 100644 --- a/bgpd/bgp_flowspec_vty.c +++ b/bgpd/bgp_flowspec_vty.c @@ -294,7 +294,7 @@ void route_vty_out_flowspec(struct vty *vty, struct prefix *p, } if (!path) return; - if (path->attr && path->attr->ecommunity) { + if (path->attr->ecommunity) { /* Print attribute */ attr = path->attr; s = ecommunity_ecom2str(attr->ecommunity, |