o Fixed a bug which caused -PN to erronously bail out for unprivileged

users.  Thanks to Jabra (jabra(a)spl0it.org) for the report. [Kris]

This was introduced when the ping probes were reorganized and moved
out of NmapOps (the o.pingtype check wasn't copied over)
This commit is contained in:
kris 2008-06-10 05:16:29 +00:00
parent 4eb6b3b5c4
commit 1d0456acad
2 changed files with 4 additions and 1 deletions

View file

@ -26,6 +26,9 @@ o Fixed the nmap_command_path bug in Zenmap. The variable now actually
by a new class in UmitConf called PathsConfig, which represents the
[paths] section in zenmap.conf. [Jurand Nogiec]
o Fixed a bug which caused -PN to erronously bail out for unprivileged
users. Thanks to Jabra (jabra(a)spl0it.org) for the report. [Kris]
o Fixed an integer overflow which prevented a target specification
of "*.*.*.*" from working. Support for the CIDR /0 is now also
available. [Kris]

View file

@ -468,7 +468,7 @@ void validate_scan_lists(scan_lists &ports, NmapOps &o){
assert(ports.ack_ping_count > 0);
}
if (!o.isr00t || o.pf() != PF_INET) {
if ((o.pingtype & PINGTYPE_TCP) && (!o.isr00t || o.pf() != PF_INET)) {
// We will have to do a connect() style ping
if (ports.syn_ping_count && ports.ack_ping_count) {
fatal("Cannot use both SYN and ACK ping probes if you are nonroot or using IPv6");