Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issues with neighbors with same link local address #68

Closed
XDjackieXD opened this issue Jun 5, 2024 · 1 comment
Closed

Issues with neighbors with same link local address #68

XDjackieXD opened this issue Jun 5, 2024 · 1 comment

Comments

@XDjackieXD
Copy link

XDjackieXD commented Jun 5, 2024

This might be intended behaviour but I'm unsure.
At least this issue should serve as documentation should someone else stumble upon this.

When a node has multiple neighbours with the same link local address (for example we had a wireguard tunnel server and were using a staticly configured LL address on the wireguard interface that was the same on all other nodes), OLSRD2 treats these neighbours as the same neighbour even though their advertised IP address and interface is different.

In our case this resulted in only a single route that was the IP address of the last node that was connected routed via the interface of the first node that was connected.
Changing the LL addresses of all nodes to be unique to each node solved this issue (having the same LL on all interfaces of one node is ok though).

For me this looks like a bug where the OLSRD2 does only look at the LL IP to identify a neighbour without looking at the interface name of this LL address (purely a guess).
I don't think that a fix for this has a high priority but awareness for this behaviour might be important ^^

@mathiashro
Copy link
Contributor

As this software is currently not beeing further developed I would keep this case as documentation for future reference and close it for now. Can be reopened at any later point of time.

@XDjackieXD , if you plan to provide a more depper analysis and if needed patch / PR - go for it.

@mathiashro mathiashro closed this as not planned Won't fix, can't repro, duplicate, stale Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants