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

Ignore simple ping #5

Open
unixfox opened this issue Nov 13, 2015 · 5 comments
Open

Ignore simple ping #5

unixfox opened this issue Nov 13, 2015 · 5 comments

Comments

@unixfox
Copy link

unixfox commented Nov 13, 2015

If a machine ping me during the program is launched, icmptunnel crashed with this error:

root@debian:~/icmptunnel# ./icmptunnel -s 10.0.1.1
[DEBUG] Allocatating tunnel
[DEBUG] Allocatating tunnel2[DEBUG] Created tunnel tun0
[DEBUG] Starting tunnel - Dest: 10.0.1.1, Server: 1
[DEBUG] Opening ICMP socket
[DEBUG] Binding ICMP socket
0+1 records in
0+1 records out
2 bytes (2 B) copied, 9.6663e-05 s, 20.7 kB/s
0+1 records in
0+1 records out
2 bytes (2 B) copied, 7.4032e-05 s, 27.0 kB/s
[DEBUG] Script ran successfully
[DEBUG] Received ICMP packet
[DEBUG] Read ICMP packet with src: 194.90.37.75, dest: 94.23.11.59, payload_size: 129, payload: E
[DEBUG] Writing to tunnel
[DEBUG] Src address being copied: 194.90.37.75
[DEBUG] Received ICMP packet
[DEBUG] Read ICMP packet with src: 188.235.15.178, dest: 94.23.11.59, payload_size: 129, payload: Ep
[DEBUG] Writing to tunnel
[DEBUG] Src address being copied: 188.235.15.178
[DEBUG] Received ICMP packet
[DEBUG] Read ICMP packet with src: 92.222.185.1, dest: 94.23.11.59, payload_size: 4, payload: ^�
;
[DEBUG] Writing to tunnel
Unable to write to tunnel
: Invalid argument

@DhavalKapil
Copy link
Owner

I was thinking of somehow making my ICMP packets unique. So that I could filter out my own packets.

@giuscri
Copy link

giuscri commented Dec 30, 2015

making my ICMP packets unique

What about starting the data wrapped in the ICMP packet with some magic bytes?

Say, b'ICMPTUN'?

@DhavalKapil
Copy link
Owner

Yeah that's what I will have to do in the end. I'll work further on this after a month. Need to take care of a lot of things which will require proper designing.

@yunfan
Copy link

yunfan commented Apr 2, 2019

@DhavalKapil please dont add unique feature, in that case it will also be identified by those firewall maker. i think just fix the wrong data problem might be okay

@fkerem
Copy link

fkerem commented Mar 14, 2021

I have the same issue, any updates?

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

5 participants