diff options
author | John Lo <loj@cisco.com> | 2018-06-19 15:27:48 -0400 |
---|---|---|
committer | Dave Barach <openvpp@barachs.net> | 2018-06-19 21:26:10 +0000 |
commit | 084606befb45507e9241b5555f86196fe08530f6 (patch) | |
tree | 25982dd871832335b2f62159657c3b3b31e361f2 /src/plugins/cdp | |
parent | 7e607a73898b81c21a24038a2350fdda3b1c67ff (diff) |
Check get packet template allocation failure (VPP-1321)
After calling vlib_packet_template_get_packet(), make sure
packet buffer is allocated before using it.
Change-Id: Idb5199f4e2c9596137b2101e502d611f474a6ffe
Signed-off-by: John Lo <loj@cisco.com>
Diffstat (limited to 'src/plugins/cdp')
-rw-r--r-- | src/plugins/cdp/cdp_periodic.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/plugins/cdp/cdp_periodic.c b/src/plugins/cdp/cdp_periodic.c index c67af00c7a5..a4c4bb91765 100644 --- a/src/plugins/cdp/cdp_periodic.c +++ b/src/plugins/cdp/cdp_periodic.c @@ -195,6 +195,9 @@ send_hdlc_hello (cdp_main_t * cm, cdp_neighbor_t * n, int count) h0 = vlib_packet_template_get_packet (vm, &cm->packet_templates[n->packet_template_index], &bi0); + if (!h0) + break; + hw = vnet_get_sup_hw_interface (vnm, n->sw_if_index); t0 = (u8 *) & h0->cdp.data; @@ -251,6 +254,9 @@ send_srp_hello (cdp_main_t * cm, cdp_neighbor_t * n, int count) h0 = vlib_packet_template_get_packet (vm, &cm->packet_templates[n->packet_template_index], &bi0); + if (!h0) + break; + hw = vnet_get_sup_hw_interface (vnm, n->sw_if_index); t0 = (u8 *) & h0->cdp.data; |