From: Ondrej Zajicek (work) <santiago@crfreenet.org>
Date: Thu, 25 Oct 2018 15:22:37 +0000 (+0200)
Subject: OSPF: Fix some trace messages
X-Git-Tag: v2.0.3~45
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=0ac9cb2c1f6592290e025f61ccd0fef0dc09de46;p=thirdparty%2Fbird.git

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.
---

diff --git a/proto/ospf/hello.c b/proto/ospf/hello.c
index 523c24add..9e427c4f7 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 19bcfa15b..54a355b6b 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);