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

Broken ping to local interfaces with IPv6 #99

Closed
pevik opened this issue Sep 14, 2017 · 1 comment
Closed

Broken ping to local interfaces with IPv6 #99

pevik opened this issue Sep 14, 2017 · 1 comment

Comments

@pevik
Copy link
Contributor

pevik commented Sep 14, 2017

This bug was introduced in #57.

$ ip addr |grep 'inet6'
    inet6 fe80::fc54:ff:fe2d:cc21/64 scope link

$ ./ping -6 fe80::fc54:ff:fe2d:cc21
connect: Invalid argument
pevik added a commit to pevik/iputils that referenced this issue Sep 14, 2017
This reverts commit e25568f.

as it breaks ping to local IPv6 interfaces:

$ ip addr |grep 'inet6'
    inet6 fe80::fc54:ff:fe2d:cc21/64 scope link

$ ./ping -6 fe80::fc54:ff:fe2d:cc21
connect: Invalid argument

Fixes iputils#99 (regression in iputils#57).
pevik added a commit to pevik/iputils that referenced this issue Sep 16, 2017
This reverts commit e25568f.

as it breaks ping to local IPv6 interfaces:
This is a regression from e25568f ("correctly initialize first hop")

$ ip -6 addr |grep 'inet6'
    inet6 fe80::fc54:ff:fe2d:cc21/64 scope link
$ ./ping -6 fe80::fc54:ff:fe2d:cc21
connect: Invalid argument

$ ip -6 maddr |grep 'inet6'
	inet6 ff02::1
$ ./ping -6 ff02::1
connect: Invalid argument

Fixes iputils#99 (regression in iputils#57).
pevik added a commit to pevik/iputils that referenced this issue Sep 16, 2017
…ication

link-local address required to specify interface either with -I option
option) or '%'-notation in destination.
If not it failed with "connect: Invalid argument".
Now it's possible to ping without it.

NOTE: it used to be possible in the past: added in f68eec0 ("ping:
perform dual-stack ping by default") and removed again in
e25568f ("correctly initialize first hop").

+ allow this for link-local scope multicast address (TODO: is it
convinient?)

Fixes iputils#99.
pevik added a commit to pevik/iputils that referenced this issue Oct 30, 2017
This reverts commit e25568f.

as it breaks ping to local IPv6 interfaces:

$ ip addr |grep 'inet6'
    inet6 fe80::fc54:ff:fe2d:cc21/64 scope link

$ ./ping -6 fe80::fc54:ff:fe2d:cc21
connect: Invalid argument

Fixes iputils#99 (regression in iputils#57).
@pevik
Copy link
Contributor Author

pevik commented Nov 1, 2017

NOTE: it used to be possible in the past: added in f68eec0 ("ping: perform dual-stack ping by default") and removed again in e25568f ("correctly initialize first hop").

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

1 participant