Project

General

Profile

Bug #2554

bypasslan creates passthrough for route with nexthop set

Added by Noel Kuntze almost 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Category:
kernel-interface
Target version:
Start date:
Due date:
Estimated time:
Affected version:
5.6.2
Resolution:
Fixed

Description

The bypasslan plugin installs passthroughs for routes that are not for directly attached networks. It shouldn't do that.

swanctl -P

[...]
bypass-lan/Bypass LAN 52.29.117.25/32, PASS
  local:  52.29.117.25/32                  
  remote: 52.29.117.25/32  
[...]

ip r

default via 192.168.178.1 dev br0 
52.29.117.25 via 192.168.178.65 dev br0 mtu lock 1300 
[...]

Associated revisions

Revision 9189aec6 (diff)
Added by Tobias Brunner almost 3 years ago

kernel-netlink: Ignore routes with next hop during local subnet enumeration

These are not locally attached and we do the same already in kernel-pfroute.

Fixes #2554.

History

#1 Updated by Tobias Brunner almost 3 years ago

  • Tracker changed from Issue to Bug
  • Category set to kernel-interface
  • Status changed from New to Feedback
  • Assignee set to Tobias Brunner
  • Target version set to 5.6.3

Agreed. I noticed that I explicitly excluded routes with RTF_GATEWAY in the kernel-pfroute plugin, so makes sense to do the same in the kernel-netlink plugin.

I pushed a fix to the 2554-local-lan-gateway branch. Let me know if this works for you.

#2 Updated by Noel Kuntze almost 3 years ago

That solves this issue. I found one/two more. See #2555.

#3 Updated by Tobias Brunner almost 3 years ago

  • Status changed from Feedback to Closed
  • Resolution set to Fixed

Also available in: Atom PDF