diff options
| author | Antonio Quartulli <antonio@openvpn.net> | 2025-05-09 15:32:13 +0200 |
|---|---|---|
| committer | Antonio Quartulli <antonio@openvpn.net> | 2025-05-15 13:09:36 +0200 |
| commit | 0ca74dfabdfe9c6274b11554adc46b79d6f44955 (patch) | |
| tree | d4d53e53e32e191be72e3fe8ff95e3b1ec8fafa0 | |
| parent | adcdaac57d3ccb38f2f1b0a8da31b5c1403385f0 (diff) | |
ovpn: improve 'no route to host' debug message
When debugging a 'no route to host' error it can be beneficial
to know the address of the unreachable destination.
Print it along the debugging text.
While at it, add a missing parenthesis in a different debugging
message inside ovpn_peer_endpoints_update().
Signed-off-by: Antonio Quartulli <antonio@openvpn.net>
| -rw-r--r-- | drivers/net/ovpn/io.c | 14 | ||||
| -rw-r--r-- | drivers/net/ovpn/peer.c | 2 |
2 files changed, 13 insertions, 3 deletions
diff --git a/drivers/net/ovpn/io.c b/drivers/net/ovpn/io.c index 43f428ac112e6..10d8afecec553 100644 --- a/drivers/net/ovpn/io.c +++ b/drivers/net/ovpn/io.c @@ -394,8 +394,18 @@ netdev_tx_t ovpn_net_xmit(struct sk_buff *skb, struct net_device *dev) /* retrieve peer serving the destination IP of this packet */ peer = ovpn_peer_get_by_dst(ovpn, skb); if (unlikely(!peer)) { - net_dbg_ratelimited("%s: no peer to send data to\n", - netdev_name(ovpn->dev)); + switch (skb->protocol) { + case htons(ETH_P_IP): + net_dbg_ratelimited("%s: no peer to send data to dst=%pI4\n", + netdev_name(ovpn->dev), + &ip_hdr(skb)->daddr); + break; + case htons(ETH_P_IPV6): + net_dbg_ratelimited("%s: no peer to send data to dst=%pI6c\n", + netdev_name(ovpn->dev), + &ipv6_hdr(skb)->daddr); + break; + } goto drop; } /* dst was needed for peer selection - it can now be dropped */ diff --git a/drivers/net/ovpn/peer.c b/drivers/net/ovpn/peer.c index 24eb9d81429ee..a1fd27b9c038a 100644 --- a/drivers/net/ovpn/peer.c +++ b/drivers/net/ovpn/peer.c @@ -258,7 +258,7 @@ void ovpn_peer_endpoints_update(struct ovpn_peer *peer, struct sk_buff *skb) */ if (unlikely(!ipv6_addr_equal(&bind->local.ipv6, &ipv6_hdr(skb)->daddr))) { - net_dbg_ratelimited("%s: learning local IPv6 for peer %d (%pI6c -> %pI6c\n", + net_dbg_ratelimited("%s: learning local IPv6 for peer %d (%pI6c -> %pI6c)\n", netdev_name(peer->ovpn->dev), peer->id, &bind->local.ipv6, &ipv6_hdr(skb)->daddr); |