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.

Sign up for GitHub

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

Jump to bottom

arp: flush ARP cache when disconnecting or changing address #543

Closed
Dirbaio opened this issue Oct 6, 2021 · 1 comment
Closed

arp: flush ARP cache when disconnecting or changing address #543

Dirbaio opened this issue Oct 6, 2021 · 1 comment
Labels
kind/bug

Comments

@Dirbaio
Copy link
Member

Dirbaio commented Oct 6, 2021

ARP cache should be flushed when

  1. Our IP address changes
  2. The interface goes down

Otherwise, the device may get plugged in into a network where eg router is still 192.168.1.1, but is another MAC addr. Communication might not work until the ARP cache entry expires.
deantvv added a commit to deantvv/smoltcp that referenced this issue Oct 31, 2021
@deantvv
arp: flush neighbor cache after IP update
a666a7a 
Related to smoltcp-rs#543
@deantvv deantvv mentioned this issue Oct 31, 2021
Merged
bors bot added a commit that referenced this issue Nov 3, 2021
@bors @deantvv
Merge #564
ec6f2e0 
564: arp: flush neighbor cache after IP update r=Dirbaio a=deantvv

Related to #543

Co-authored-by: Dean Li <[email protected]>
bors bot added a commit that referenced this issue Nov 3, 2021
@bors @deantvv
Merge #564
7dd009d 
564: arp: flush neighbor cache after IP update r=Dirbaio a=deantvv

Related to #543

Co-authored-by: Dean Li <[email protected]>
@elpiel
Copy link

elpiel commented Sep 30, 2023

I suppose only the second point of this issue remains to be fixed after #564 was merged?

@Dirbaio Dirbaio closed this as completed Sep 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Dirbaio @elpiel