From 50e049db76573210e3b9ccfa962e5259452ccbe3 Mon Sep 17 00:00:00 2001 From: Donald Sharp Date: Thu, 16 Mar 2023 10:24:25 -0400 Subject: [PATCH] bgpd: Use interface name instead of pointer value Log message is borked in a manner that makes it unusable: bgpd[52]: [VX6SM-8YE5W][EC 33554460] 2000:31:0:53::2: nexthop_set failed, resetting connection - intf 0x561eb9005a30 Let's print out the interface name instead. Signed-off-by: Donald Sharp --- bgpd/bgp_network.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/bgpd/bgp_network.c b/bgpd/bgp_network.c index 1c2e686e1c..6f035358f1 100644 --- a/bgpd/bgp_network.c +++ b/bgpd/bgp_network.c @@ -805,9 +805,12 @@ int bgp_getsockname(struct peer *peer) if (!bgp_zebra_nexthop_set(peer->su_local, peer->su_remote, &peer->nexthop, peer)) { - flog_err(EC_BGP_NH_UPD, - "%s: nexthop_set failed, resetting connection - intf %p", - peer->host, peer->nexthop.ifp); + flog_err( + EC_BGP_NH_UPD, + "%s: nexthop_set failed, resetting connection - intf %s", + peer->host, + peer->nexthop.ifp ? peer->nexthop.ifp->name + : "(Unknown)"); return -1; } return 0; -- 2.39.5