From 4a4051ee2c481b4a0b13a5048312e5fa188ec442 Mon Sep 17 00:00:00 2001 From: luis Date: Fri, 21 May 2010 09:46:08 +0000 Subject: [PATCH] Added note to investigate a bug I've just discovered, running one of the tests designed by David --- todo/nping.txt | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) diff --git a/todo/nping.txt b/todo/nping.txt index bfc485d54..587cbe84b 100644 --- a/todo/nping.txt +++ b/todo/nping.txt @@ -206,6 +206,33 @@ around strtoul). There is nowhere that isNumber_u* is called without it being immediately followed by a strtoul, outside of utils.cc. +* Investigate and fix a bug: nsi_pcap_linktype(), nping.cc:1717, returns + something different than DLT_EN10MB when sending UDP packets with the same + source and destination port number. If(dport==sport) it breaks; + This happens also in TCP mode, and even in ICMP mode. This is weird. It might + have something to do with the BPF filter. Some examples: + + luis@Aberdeen:~$ sudo nping --udp 163.117.131.31 -p53 + Starting Nping 0.5.30BETA1 ( http://nmap.org/nping ) at 2010-05-21 11:36 CEST + SENT (0.1140s) UDP 163.117.203.225:53 > 163.117.131.31:53 ttl=64 id=23488 iplen=28 + Currently only Ethernet is supported. + ----- + luis@Aberdeen:~$ sudo nping --udp 163.117.131.31 -p59 -g 59 + Starting Nping 0.5.30BETA1 ( http://nmap.org/nping ) at 2010-05-21 11:36 CEST + SENT (0.1060s) UDP 163.117.203.225:59 > 163.117.131.31:59 ttl=64 id=57091 iplen=28 + Currently only Ethernet is supported. + ----- + luis@Aberdeen:~$ sudo nping --tcp 163.117.131.31 -p53 -g53 + Starting Nping 0.5.30BETA1 ( http://nmap.org/nping ) at 2010-05-21 11:37 CEST + SENT (0.1070s) TCP 163.117.203.225:53 > 163.117.131.31:53 S ttl=64 id=29612 iplen=40 seq=4244174393 win=1480 + Currently only Ethernet is supported. + ----- + luis@Aberdeen:~$ sudo nping 163.117.131.31 -p53 -g53 --icmp + Starting Nping 0.5.30BETA1 ( http://nmap.org/nping ) at 2010-05-21 11:37 CEST + SENT (0.1060s) ICMP 163.117.203.225 > 163.117.131.31 Echo request (type=8/code=0) ttl=64 id=23394 iplen=28 + Currently only Ethernet is supported. + + /***************************************************************************** * Things that have been solved already * *****************************************************************************/