From 0ac9cb2c1f6592290e025f61ccd0fef0dc09de46 Mon Sep 17 00:00:00 2001 From: "Ondrej Zajicek (work)" Date: Thu, 25 Oct 2018 17:22:37 +0200 Subject: [PATCH] OSPF: Fix some trace messages Missing argument in MTU change trace message can crash bird when MTU change happens and trace messages are active. Thanks to Alexander Velkov for the bugreport. --- proto/ospf/hello.c | 2 +- proto/ospf/iface.c | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/proto/ospf/hello.c b/proto/ospf/hello.c index 523c24ad..9e427c4f 100644 --- a/proto/ospf/hello.c +++ b/proto/ospf/hello.c @@ -290,7 +290,7 @@ ospf_receive_hello(struct ospf_packet *pkt, struct ospf_iface *ifa, if (!ipa_equal(faddr, n->ip)) { OSPF_TRACE(D_EVENTS, "Neighbor %R on %s changed IP address to %I", - n->rid, ifa->ifname, n->ip, faddr); + n->rid, ifa->ifname, faddr); n->ip = faddr; } } diff --git a/proto/ospf/iface.c b/proto/ospf/iface.c index 19bcfa15..54a355b6 100644 --- a/proto/ospf/iface.c +++ b/proto/ospf/iface.c @@ -1278,7 +1278,8 @@ ospf_iface_change_mtu(struct ospf_proto *p, struct ospf_iface *ifa) { /* ifa is not vlink */ - OSPF_TRACE(D_EVENTS, "Interface %s changed MTU to %d", ifa->iface->mtu); + OSPF_TRACE(D_EVENTS, "Interface %s changed MTU to %d", + ifa->ifname, ifa->iface->mtu); ifa->tx_length = ifa_tx_length(ifa);