mirror of
https://git.proxmox.com/git/mirror_frr
synced 2025-08-03 00:13:40 +00:00
Merge pull request #4861 from NaveenThanikachalam/logs
BGP: Rectifying the log messages.
This commit is contained in:
commit
ef5307f23f
@ -1685,8 +1685,8 @@ int bgp_mp_reach_parse(struct bgp_attr_parser_args *args,
|
||||
#define BGP_MP_REACH_MIN_SIZE 5
|
||||
#define LEN_LEFT (length - (stream_get_getp(s) - start))
|
||||
if ((length > STREAM_READABLE(s)) || (length < BGP_MP_REACH_MIN_SIZE)) {
|
||||
zlog_info("%s: %s sent invalid length, %lu", __func__,
|
||||
peer->host, (unsigned long)length);
|
||||
zlog_info("%s: %s sent invalid length, %lu, of MP_REACH_NLRI",
|
||||
__func__, peer->host, (unsigned long)length);
|
||||
return BGP_ATTR_PARSE_ERROR_NOTIFYPLS;
|
||||
}
|
||||
|
||||
@ -1702,7 +1702,7 @@ int bgp_mp_reach_parse(struct bgp_attr_parser_args *args,
|
||||
*/
|
||||
if (bgp_debug_update(peer, NULL, NULL, 0))
|
||||
zlog_debug(
|
||||
"%s: MP_REACH received AFI %s or SAFI %s is unrecognized",
|
||||
"%s sent unrecognizable AFI, %s or, SAFI, %s, of MP_REACH_NLRI",
|
||||
peer->host, iana_afi2str(pkt_afi),
|
||||
iana_safi2str(pkt_safi));
|
||||
return BGP_ATTR_PARSE_ERROR;
|
||||
@ -1713,7 +1713,7 @@ int bgp_mp_reach_parse(struct bgp_attr_parser_args *args,
|
||||
|
||||
if (LEN_LEFT < attr->mp_nexthop_len) {
|
||||
zlog_info(
|
||||
"%s: %s, MP nexthop length, %u, goes past end of attribute",
|
||||
"%s: %s sent next-hop length, %u, in MP_REACH_NLRI which goes past the end of attribute",
|
||||
__func__, peer->host, attr->mp_nexthop_len);
|
||||
return BGP_ATTR_PARSE_ERROR_NOTIFYPLS;
|
||||
}
|
||||
@ -1722,7 +1722,7 @@ int bgp_mp_reach_parse(struct bgp_attr_parser_args *args,
|
||||
switch (attr->mp_nexthop_len) {
|
||||
case 0:
|
||||
if (safi != SAFI_FLOWSPEC) {
|
||||
zlog_info("%s: (%s) Wrong multiprotocol next hop length: %d",
|
||||
zlog_info("%s: %s sent wrong next-hop length, %d, in MP_REACH_NLRI",
|
||||
__func__, peer->host, attr->mp_nexthop_len);
|
||||
return BGP_ATTR_PARSE_ERROR_NOTIFYPLS;
|
||||
}
|
||||
@ -1754,7 +1754,7 @@ int bgp_mp_reach_parse(struct bgp_attr_parser_args *args,
|
||||
stream_get(&attr->mp_nexthop_global, s, IPV6_MAX_BYTELEN);
|
||||
if (IN6_IS_ADDR_LINKLOCAL(&attr->mp_nexthop_global)) {
|
||||
if (!peer->nexthop.ifp) {
|
||||
zlog_warn("%s: Received a V6/VPNV6 Global attribute but address is a V6 LL and we have no peer interface information, withdrawing",
|
||||
zlog_warn("%s sent a v6 global attribute but address is a V6 LL and there's no peer interface information. Hence, withdrawing",
|
||||
peer->host);
|
||||
return BGP_ATTR_PARSE_WITHDRAW;
|
||||
}
|
||||
@ -1771,7 +1771,7 @@ int bgp_mp_reach_parse(struct bgp_attr_parser_args *args,
|
||||
stream_get(&attr->mp_nexthop_global, s, IPV6_MAX_BYTELEN);
|
||||
if (IN6_IS_ADDR_LINKLOCAL(&attr->mp_nexthop_global)) {
|
||||
if (!peer->nexthop.ifp) {
|
||||
zlog_warn("%s: Received V6/VPNV6 Global and LL attribute but global address is a V6 LL and we have no peer interface information, withdrawing",
|
||||
zlog_warn("%s sent a v6 global and LL attribute but global address is a V6 LL and there's no peer interface information. Hence, withdrawing",
|
||||
peer->host);
|
||||
return BGP_ATTR_PARSE_WITHDRAW;
|
||||
}
|
||||
@ -1789,7 +1789,7 @@ int bgp_mp_reach_parse(struct bgp_attr_parser_args *args,
|
||||
|
||||
if (bgp_debug_update(peer, NULL, NULL, 1))
|
||||
zlog_debug(
|
||||
"%s rcvd nexthops %s, %s -- ignoring non-LL value",
|
||||
"%s sent next-hops %s and %s. Ignoring non-LL value",
|
||||
peer->host,
|
||||
inet_ntop(AF_INET6,
|
||||
&attr->mp_nexthop_global,
|
||||
@ -1801,21 +1801,21 @@ int bgp_mp_reach_parse(struct bgp_attr_parser_args *args,
|
||||
attr->mp_nexthop_len = IPV6_MAX_BYTELEN;
|
||||
}
|
||||
if (!peer->nexthop.ifp) {
|
||||
zlog_warn("%s: Received a V6 LL nexthop and we have no peer interface information, withdrawing",
|
||||
zlog_warn("%s sent a v6 LL next-hop and there's no peer interface information. Hence, withdrawing",
|
||||
peer->host);
|
||||
return BGP_ATTR_PARSE_WITHDRAW;
|
||||
}
|
||||
attr->nh_lla_ifindex = peer->nexthop.ifp->ifindex;
|
||||
break;
|
||||
default:
|
||||
zlog_info("%s: (%s) Wrong multiprotocol next hop length: %d",
|
||||
zlog_info("%s: %s sent wrong next-hop length, %d, in MP_REACH_NLRI",
|
||||
__func__, peer->host, attr->mp_nexthop_len);
|
||||
return BGP_ATTR_PARSE_ERROR_NOTIFYPLS;
|
||||
}
|
||||
|
||||
if (!LEN_LEFT) {
|
||||
zlog_info("%s: (%s) Failed to read SNPA and NLRI(s)", __func__,
|
||||
peer->host);
|
||||
zlog_info("%s: %s sent SNPA which couldn't be read",
|
||||
__func__, peer->host);
|
||||
return BGP_ATTR_PARSE_ERROR_NOTIFYPLS;
|
||||
}
|
||||
|
||||
@ -1831,12 +1831,13 @@ int bgp_mp_reach_parse(struct bgp_attr_parser_args *args,
|
||||
/* must have nrli_len, what is left of the attribute */
|
||||
nlri_len = LEN_LEFT;
|
||||
if (nlri_len > STREAM_READABLE(s)) {
|
||||
zlog_info("%s: (%s) Failed to read NLRI", __func__, peer->host);
|
||||
zlog_info("%s: %s sent MP_REACH_NLRI which couldn't be read",
|
||||
__func__, peer->host);
|
||||
return BGP_ATTR_PARSE_ERROR_NOTIFYPLS;
|
||||
}
|
||||
|
||||
if (!nlri_len) {
|
||||
zlog_info("%s: (%s) No Reachability, Treating as a EOR marker",
|
||||
zlog_info("%s: %s sent a zero-length NLRI. Hence, treating as a EOR marker",
|
||||
__func__, peer->host);
|
||||
|
||||
mp_update->afi = afi;
|
||||
@ -2043,8 +2044,8 @@ static int bgp_attr_encap(uint8_t type, struct peer *peer, /* IN */
|
||||
length -= 4;
|
||||
|
||||
if (tlv_length != length) {
|
||||
zlog_info("%s: tlv_length(%d) != length(%d)", __func__,
|
||||
tlv_length, length);
|
||||
zlog_info("%s: tlv_length(%d) != length(%d)",
|
||||
__func__, tlv_length, length);
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -1324,7 +1324,7 @@ static int bgp_connect_check(struct thread *thread)
|
||||
|
||||
/* If getsockopt is fail, this is fatal error. */
|
||||
if (ret < 0) {
|
||||
zlog_info("can't get sockopt for nonblocking connect: %d(%s)",
|
||||
zlog_err("can't get sockopt for nonblocking connect: %d(%s)",
|
||||
errno, safe_strerror(errno));
|
||||
BGP_EVENT_ADD(peer, TCP_fatal_error);
|
||||
return -1;
|
||||
|
@ -1031,7 +1031,7 @@ as_t peek_for_as4_capability(struct peer *peer, uint8_t length)
|
||||
as_t as4 = 0;
|
||||
|
||||
if (BGP_DEBUG(as4, AS4))
|
||||
zlog_info(
|
||||
zlog_debug(
|
||||
"%s [AS4] rcv OPEN w/ OPTION parameter len: %u,"
|
||||
" peeking for as4",
|
||||
peer->host, length);
|
||||
@ -1075,7 +1075,7 @@ as_t peek_for_as4_capability(struct peer *peer, uint8_t length)
|
||||
|
||||
if (hdr.code == CAPABILITY_CODE_AS4) {
|
||||
if (BGP_DEBUG(as4, AS4))
|
||||
zlog_info(
|
||||
zlog_debug(
|
||||
"[AS4] found AS4 capability, about to parse");
|
||||
as4 = bgp_capability_as4(peer, &hdr);
|
||||
|
||||
|
@ -2053,7 +2053,7 @@ int peer_activate(struct peer *peer, afi_t afi, safi_t safi)
|
||||
&& !bgp->allocate_mpls_labels[afi][SAFI_UNICAST]) {
|
||||
|
||||
if (BGP_DEBUG(zebra, ZEBRA))
|
||||
zlog_info(
|
||||
zlog_debug(
|
||||
"peer(s) are now active for labeled-unicast, allocate MPLS labels");
|
||||
|
||||
bgp->allocate_mpls_labels[afi][SAFI_UNICAST] = 1;
|
||||
@ -2156,7 +2156,7 @@ int peer_deactivate(struct peer *peer, afi_t afi, safi_t safi)
|
||||
&& !bgp_afi_safi_peer_exists(bgp, afi, safi)) {
|
||||
|
||||
if (BGP_DEBUG(zebra, ZEBRA))
|
||||
zlog_info(
|
||||
zlog_debug(
|
||||
"peer(s) are no longer active for labeled-unicast, deallocate MPLS labels");
|
||||
|
||||
bgp->allocate_mpls_labels[afi][SAFI_UNICAST] = 0;
|
||||
|
Loading…
Reference in New Issue
Block a user