subtype and sublength are not initialized and if on Line 1877 BGP_ATTR_ENCAP != type we will not set subtype and sublength, but these variables are used immediately below that if statement.
This issue was discovered via the freebsd compiler. Signed-off-by: Donald Sharp <sha...@cumulusnetworks.com> --- bgpd/bgp_attr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bgpd/bgp_attr.c b/bgpd/bgp_attr.c index f34e649..149b888 100644 --- a/bgpd/bgp_attr.c +++ b/bgpd/bgp_attr.c @@ -1870,8 +1870,8 @@ bgp_attr_encap( } while (length >= 4) { - uint16_t subtype; - uint16_t sublength; + uint16_t subtype = 0; + uint16_t sublength = 0; struct bgp_attr_encap_subtlv *tlv; if (BGP_ATTR_ENCAP == type) { -- 1.9.1 _______________________________________________ Quagga-dev mailing list Quagga-dev@lists.quagga.net https://lists.quagga.net/mailman/listinfo/quagga-dev