This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Update Net-Ping to CPAN version 2.51
authorChris 'BinGOs' Williams <chris@bingosnet.co.uk>
Tue, 18 Oct 2016 10:14:18 +0000 (11:14 +0100)
committerChris 'BinGOs' Williams <chris@bingosnet.co.uk>
Tue, 18 Oct 2016 11:40:03 +0000 (12:40 +0100)
  [DELTA]

2.51  Mon Oct 17 16:11:03 2016 +0200 (rurban)
       version in cperl since 5.25.2c

       Bugfixes
       - Fixed missing _unpack_sockaddr_in family, which took AF_INET6 for
         a AF_INET addr in t/500_ping_icmp.t and t/500_ping_icmp_ttl.t.
         Use now a proper default.

2.50  Sat Apr 16 11:50:20 2016 +0200 (rurban)
       version in cperl since 5.22.2c

       Features
       - Handle IPv6 addresses and the AF_INET6 family.
       - Added the optional family argument to most methods.
         valid values: 6, "v6", "ip6", "ipv6", AF_INET6
       - new can take now named arguments, a hashref.
       - Added the following named arguments to new:
         gateway host port bind retrans pingstring source_verify econnrefused
         IPV6_USE_MIN_MTU IPV6_RECVPATHMTU IPV6_HOPLIMIT
       - Added a dontfrag option, setting IP_DONTFRAG and on linux
         also IP_MTU_DISCOVER to IP_PMTUDISC_DO. Note that is ignored if
         Socket does not export IP_DONTFRAG.
       - Added the wakeonlan method
       - Improve argument default handling
       - Added missing documentation

       Bugfixes
       - Reapply tos with ping_udp, when the address is changed.
         RT #6706 (Torgny.Hofstedt@sevenlevels.se)
         ditto re-bind to a device.

       Internals
       - $ip is now a hash with {addr, addr_in, family} not the addr_in packed IP.
       - added _resolv replacing inet_aton,
         _pack_sockaddr_in and _unpack_sockaddr_in replacing sockaddr_in,
         _inet_ntoa replacing inet_ntoa
       - Use _isroot helper, with Win32 _IsAdminUser helper.
       - added several new tests (Steve Peters)

2.43  Mon Apr 29 00:23:56 2013 -0300
        version in perl core since 5.19.9
        Bugfixes
        - Handle getprotobyn{ame,umber} not being available
2.42  Sun May 26 19:08:46 2013 -0700
        version in perl core since 5.19.1
        Bugfixes
        - Stabilize tests
       Internals
        - wrap long pod lines

12 files changed:
MANIFEST
Porting/Maintainers.pl
dist/Net-Ping/Changes
dist/Net-Ping/lib/Net/Ping.pm
dist/Net-Ping/t/000_load.t [new file with mode: 0644]
dist/Net-Ping/t/001_new.t [new file with mode: 0644]
dist/Net-Ping/t/010_pingecho.t [new file with mode: 0644]
dist/Net-Ping/t/100_load.t [deleted file]
dist/Net-Ping/t/110_icmp_inst.t
dist/Net-Ping/t/500_ping_icmp.t
dist/Net-Ping/t/520_icmp_ttl.t
t/porting/known_pod_issues.dat

index 8d3d0f1..0a895d7 100644 (file)
--- a/MANIFEST
+++ b/MANIFEST
@@ -3468,7 +3468,9 @@ dist/Module-CoreList/t/pod.t                      Module::CoreList tests
 dist/Module-CoreList/t/utils.t                 Module::CoreList tests
 dist/Net-Ping/Changes                  Net::Ping
 dist/Net-Ping/lib/Net/Ping.pm          Hello, anybody home?
-dist/Net-Ping/t/100_load.t             Ping Net::Ping
+dist/Net-Ping/t/000_load.t
+dist/Net-Ping/t/001_new.t
+dist/Net-Ping/t/010_pingecho.t
 dist/Net-Ping/t/110_icmp_inst.t                Ping Net::Ping
 dist/Net-Ping/t/120_udp_inst.t         Ping Net::Ping
 dist/Net-Ping/t/130_tcp_inst.t         Ping Net::Ping
index 3a097af..e92fab2 100755 (executable)
@@ -876,8 +876,14 @@ use File::Glob qw(:case);
     },
 
     'Net::Ping' => {
-        'DISTRIBUTION' => 'SMPETERS/Net-Ping-2.41.tar.gz',
+        'DISTRIBUTION' => 'RURBAN/Net-Ping-2.51.tar.gz',
         'FILES'        => q[dist/Net-Ping],
+        'EXCLUDED'     => [
+            qw(t/020_external.t),
+            qw(t/600_pod.t),
+            qw(t/601_pod-coverage.t),
+        ],
+
     },
 
     'NEXT' => {
index fa26c68..2251724 100644 (file)
@@ -1,5 +1,54 @@
 CHANGES
 -------
+2.51  Mon Oct 17 16:11:03 2016 +0200 (rurban)
+       version in cperl since 5.25.2c
+
+       Bugfixes
+       - Fixed missing _unpack_sockaddr_in family, which took AF_INET6 for
+         a AF_INET addr in t/500_ping_icmp.t and t/500_ping_icmp_ttl.t.
+         Use now a proper default.
+
+2.50  Sat Apr 16 11:50:20 2016 +0200 (rurban)
+       version in cperl since 5.22.2c
+
+       Features
+       - Handle IPv6 addresses and the AF_INET6 family.
+       - Added the optional family argument to most methods.
+         valid values: 6, "v6", "ip6", "ipv6", AF_INET6
+       - new can take now named arguments, a hashref.
+       - Added the following named arguments to new:
+         gateway host port bind retrans pingstring source_verify econnrefused
+         IPV6_USE_MIN_MTU IPV6_RECVPATHMTU IPV6_HOPLIMIT
+       - Added a dontfrag option, setting IP_DONTFRAG and on linux
+         also IP_MTU_DISCOVER to IP_PMTUDISC_DO. Note that is ignored if
+         Socket does not export IP_DONTFRAG.
+       - Added the wakeonlan method
+       - Improve argument default handling
+       - Added missing documentation
+
+       Bugfixes
+       - Reapply tos with ping_udp, when the address is changed.
+         RT #6706 (Torgny.Hofstedt@sevenlevels.se)
+         ditto re-bind to a device.
+
+       Internals
+       - $ip is now a hash with {addr, addr_in, family} not the addr_in packed IP.
+       - added _resolv replacing inet_aton,
+         _pack_sockaddr_in and _unpack_sockaddr_in replacing sockaddr_in,
+         _inet_ntoa replacing inet_ntoa
+       - Use _isroot helper, with Win32 _IsAdminUser helper.
+       - added several new tests (Steve Peters)
+
+2.43  Mon Apr 29 00:23:56 2013 -0300
+        version in perl core since 5.19.9
+        Bugfixes
+        - Handle getprotobyn{ame,umber} not being available
+2.42  Sun May 26 19:08:46 2013 -0700
+        version in perl core since 5.19.1
+        Bugfixes
+        - Stabilize tests
+       Internals
+        - wrap long pod lines
 2.41  Mar 17 09:35 2013
         Bugfixes
         - Windows Vista does not appear to support inet_ntop().  It seems to
@@ -7,31 +56,31 @@ CHANGES
           and passing in the NI_NUMERICHOST to get an IP address.
         Features
         - Change Net::Ping to use Time::HiRes::time() instead of CORE::time()
-          by default.  For most successful cases, CORE::time() returned zero.
+          by default.  For most successful cases, CORE::time() returned zero. 
 2.40  Mar 15 11:20 2013
         Bugfixes
-        - several fixes to tests to stop the black smoke on Win32's
+        - several fixes to tests to stop the black smoke on Win32's 
           and Cygwin since the core updated the module to Test::More.
           I had planned a later release, but all the black smoke is
           forcing a release.
-        - fixes to some skips in tests that were still using the
+        - fixes to some skips in tests that were still using the 
           Test style skip's.
         - Documentation fix for https://rt.cpan.org/Ticket/Display.html?id=48014.
           Thanks to Keith Taylor <keith@supanet.net.uk>
-        - Instead of using a hard-coded TOS value, import IP_TOS from
-          Socket.  This fixes an outstanding bug on Solaris which uses a
+        - Instead of using a hard-coded TOS value, import IP_TOS from 
+          Socket.  This fixes an outstanding bug on Solaris which uses a 
           different value for IP_TOS in it headers than Linux.  I'm assuming
           other OS's were fixed with this change as well.
 
         Features
-        - added TTL handling for icmp pings to allow traceroute like
-          applications to be built with Net::Ping.  Thanks to
+        - added TTL handling for icmp pings to allow traceroute like 
+          applications to be built with Net::Ping.  Thanks to 
           <rolek@bokxing.nl> for the patch and tests!
 
        Internals
-        - replaced SOL_IP with IPPROTO_IP.  SOL_IP is not portable and was
+        - replaced SOL_IP with IPPROTO_IP.  SOL_IP is not portable and was 
           hard-coded anyway.
-        - added IPPROTO_IP, IP_TOS, IP_TTL, and AF_INET to the list of Socket
+        - added IPPROTO_IP, IP_TOS, IP_TTL, and AF_INET to the list of Socket 
           constants imported.
         - removed some hard-coded constants.
         - converted all calls to inet_ntoa() to inet_ntop() in preparation
@@ -56,7 +105,7 @@ CHANGES
         - release to include a few fixes from the Perl core
 
 2.35  Feb 08 14:42 2008
-       - Patch in Perl change #33242 by Nicholas Clark
+       - Patch in Perl change #33242 by Nicholas Clark 
                <http://perl5.git.perl.org/perl.git/commit/5d6b07c5a4c042580b85248d570ee299fd102a79>
 
 2.34  Dec 19 08:51 2007
index 73d2a83..bad39f9 100644 (file)
@@ -4,32 +4,48 @@ require 5.002;
 require Exporter;
 
 use strict;
-use vars qw(@ISA @EXPORT $VERSION
-            $def_timeout $def_proto $def_factor
+use vars qw(@ISA @EXPORT @EXPORT_OK $VERSION
+            $def_timeout $def_proto $def_factor $def_family
             $max_datasize $pingstring $hires $source_verify $syn_forking);
 use Fcntl qw( F_GETFL F_SETFL O_NONBLOCK );
-use Socket qw( SOCK_DGRAM SOCK_STREAM SOCK_RAW PF_INET SOL_SOCKET SO_ERROR IPPROTO_IP IP_TOS IP_TTL
-               inet_aton getnameinfo NI_NUMERICHOST sockaddr_in );
-use POSIX qw( ENOTCONN ECONNREFUSED ECONNRESET EINPROGRESS EWOULDBLOCK EAGAIN WNOHANG );
+use Socket qw( SOCK_DGRAM SOCK_STREAM SOCK_RAW AF_INET PF_INET IPPROTO_TCP
+              SOL_SOCKET SO_ERROR SO_BROADCAST
+               IPPROTO_IP IP_TOS IP_TTL
+               inet_ntoa inet_aton getnameinfo NI_NUMERICHOST sockaddr_in );
+use POSIX qw( ENOTCONN ECONNREFUSED ECONNRESET EINPROGRESS EWOULDBLOCK EAGAIN
+             WNOHANG );
 use FileHandle;
 use Carp;
 use Time::HiRes;
 
 @ISA = qw(Exporter);
 @EXPORT = qw(pingecho);
-$VERSION = "2.44";
+@EXPORT_OK = qw(wakeonlan);
+$VERSION = "2.51";
 
-# Constants
+# Globals
 
 $def_timeout = 5;           # Default timeout to wait for a reply
 $def_proto = "tcp";         # Default protocol to use for pinging
 $def_factor = 1.2;          # Default exponential backoff rate.
+$def_family = AF_INET;      # Default family.
 $max_datasize = 1024;       # Maximum data bytes in a packet
 # The data we exchange with the server for the stream protocol
 $pingstring = "pingschwingping!\n";
 $source_verify = 1;         # Default is to verify source endpoint
 $syn_forking = 0;
 
+# Constants
+
+my $AF_INET6  = eval { Socket::AF_INET6() };
+my $AF_UNSPEC = eval { Socket::AF_UNSPEC() };
+my $AI_NUMERICHOST = eval { Socket::AI_NUMERICHOST() };
+my $NI_NUMERICHOST = eval { Socket::NI_NUMERICHOST() };
+my $IPPROTO_IPV6   = eval { Socket::IPPROTO_IPV6() };
+#my $IPV6_HOPLIMIT  = eval { Socket::IPV6_HOPLIMIT() };  # ping6 -h 0-255
+my $qr_family = qr/^(?:(?:(:?ip)?v?(?:4|6))|${\AF_INET}|$AF_INET6)$/;
+my $qr_family4 = qr/^(?:(?:(:?ip)?v?4)|${\AF_INET})$/;
+
 if ($^O =~ /Win32/i) {
   # Hack to avoid this Win32 spewage:
   # Your vendor has not defined POSIX macro ECONNREFUSED
@@ -50,10 +66,6 @@ if ($^O =~ /Win32/i) {
 #  $syn_forking = 1;    # XXX possibly useful in < Win2K ?
 };
 
-# h2ph "asm/socket.h"
-# require "asm/socket.ph";
-sub SO_BINDTODEVICE {25;}
-
 # Description:  The pingecho() subroutine is provided for backward
 # compatibility with the original Net::Ping.  It accepts a host
 # name/IP and an optional timeout in seconds.  Create a tcp ping
@@ -86,6 +98,7 @@ sub new
       $device,            # Optional device to use
       $tos,               # Optional ToS to set
       $ttl,               # Optional TTL to set
+      $family,            # Optional address family (AF_INET)
       ) = @_;
   my  $class = ref($this) || $this;
   my  $self = {};
@@ -94,10 +107,29 @@ sub new
       );
 
   bless($self, $class);
+  if (ref $proto eq 'HASH') { # support named args
+    for my $k (qw(proto timeout data_size device tos ttl family
+                  gateway host port bind retrans pingstring source_verify
+                  econnrefused dontfrag
+                  IPV6_USE_MIN_MTU IPV6_RECVPATHMTU IPV6_HOPLIMIT))
+    {
+      if (exists $proto->{$k}) {
+        $self->{$k} = $proto->{$k};
+        # some are still globals
+        if ($k eq 'pingstring') { $pingstring = $proto->{$k} }
+        if ($k eq 'source_verify') { $source_verify = $proto->{$k} }
+        delete $proto->{$k};
+      }
+    }
+    if (%$proto) {
+      croak("Invalid named argument: ",join(" ",keys (%$proto)));
+    }
+    $proto = $self->{'proto'};
+  }
 
   $proto = $def_proto unless $proto;          # Determine the protocol
-  croak('Protocol for ping must be "icmp", "udp", "tcp", "syn", "stream", or "external"')
-    unless $proto =~ m/^(icmp|udp|tcp|syn|stream|external)$/;
+  croak('Protocol for ping must be "icmp", "icmpv6", "udp", "tcp", "syn", "stream" or "external"')
+    unless $proto =~ m/^(icmp|icmpv6|udp|tcp|syn|stream|external)$/;
   $self->{"proto"} = $proto;
 
   $timeout = $def_timeout unless $timeout;    # Determine the timeout
@@ -109,10 +141,41 @@ sub new
 
   $self->{"tos"} = $tos;
 
-  if ($self->{"proto"} eq 'icmp') {
+  if ($self->{'host'}) {
+    my $host = $self->{'host'};
+    my $ip = _resolv($host)
+      or croak("could not resolve host $host");
+    $self->{host} = $ip;
+    $self->{family} = $ip->{family};
+  }
+
+  if ($self->{bind}) {
+    my $addr = $self->{bind};
+    my $ip = _resolv($addr)
+      or croak("could not resolve local addr $addr");
+    $self->{local_addr} = $ip;
+  } else {
+    $self->{local_addr} = undef;              # Don't bind by default
+  }
+
+  if ($self->{proto} eq 'icmp') {
     croak('TTL must be from 0 to 255')
       if ($ttl && ($ttl < 0 || $ttl > 255));
-    $self->{"ttl"} = $ttl;
+    $self->{ttl} = $ttl;
+  }
+
+  if ($family) {
+    if ($family =~ $qr_family) {
+      if ($family =~ $qr_family4) {
+        $self->{"family"} = AF_INET;
+      } else {
+        $self->{"family"} = $AF_INET6;
+      }
+    } else {
+      croak('Family must be "ipv4" or "ipv6"')
+    }
+  } else {
+    $self->{"family"} = $def_family;
   }
 
   $min_datasize = ($proto eq "udp") ? 1 : 0;  # Determine data size
@@ -128,49 +191,85 @@ sub new
     $self->{"data"} .= chr($cnt % 256);
   }
 
-  $self->{"local_addr"} = undef;              # Don't bind by default
-  $self->{"retrans"} = $def_factor;           # Default exponential backoff rate
-  $self->{"econnrefused"} = undef;            # Default Connection refused behavior
+  # Default exponential backoff rate
+  $self->{"retrans"} = $def_factor unless exists $self->{"retrans"};
+  # Default Connection refused behavior
+  $self->{"econnrefused"} = undef unless exists $self->{"econnrefused"};
 
   $self->{"seq"} = 0;                         # For counting packets
   if ($self->{"proto"} eq "udp")              # Open a socket
   {
     $self->{"proto_num"} = eval { (getprotobyname('udp'))[2] } ||
       croak("Can't udp protocol by name");
-    $self->{"port_num"} = (getservbyname('echo', 'udp'))[2] ||
-      croak("Can't get udp echo port by name");
+    $self->{"port_num"} = $self->{"port"}
+      || (getservbyname('echo', 'udp'))[2]
+      || croak("Can't get udp echo port by name");
     $self->{"fh"} = FileHandle->new();
     socket($self->{"fh"}, PF_INET, SOCK_DGRAM,
            $self->{"proto_num"}) ||
              croak("udp socket error - $!");
-    if ($self->{'device'}) {
-      setsockopt($self->{"fh"}, SOL_SOCKET, SO_BINDTODEVICE(), pack("Z*", $self->{'device'}))
-        or croak "error binding to device $self->{'device'} $!";
-    }
-    if ($self->{'tos'}) {
-      setsockopt($self->{"fh"}, IPPROTO_IP, IP_TOS, pack("I*", $self->{'tos'}))
-        or croak "error configuring tos to $self->{'tos'} $!";
-    }
+    $self->_setopts();
   }
   elsif ($self->{"proto"} eq "icmp")
   {
-    croak("icmp ping requires root privilege") if ($> and $^O ne 'VMS' and $^O ne 'cygwin');
+    croak("icmp ping requires root privilege") if !_isroot();
     $self->{"proto_num"} = eval { (getprotobyname('icmp'))[2] } ||
       croak("Can't get icmp protocol by name");
     $self->{"pid"} = $$ & 0xffff;           # Save lower 16 bits of pid
     $self->{"fh"} = FileHandle->new();
     socket($self->{"fh"}, PF_INET, SOCK_RAW, $self->{"proto_num"}) ||
       croak("icmp socket error - $!");
-    if ($self->{'device'}) {
-      setsockopt($self->{"fh"}, SOL_SOCKET, SO_BINDTODEVICE(), pack("Z*", $self->{'device'}))
-        or croak "error binding to device $self->{'device'} $!";
+    $self->_setopts();
+    if ($self->{'ttl'}) {
+      setsockopt($self->{"fh"}, IPPROTO_IP, IP_TTL, pack("I*", $self->{'ttl'}))
+        or croak "error configuring ttl to $self->{'ttl'} $!";
+    }
+  }
+  elsif ($self->{"proto"} eq "icmpv6")
+  {
+    croak("icmpv6 ping requires root privilege") if !_isroot();
+    croak("Wrong family $self->{family} for icmpv6 protocol")
+      if $self->{"family"} and $self->{"family"} != $AF_INET6;
+    $self->{"family"} = $AF_INET6;
+    $self->{"proto_num"} = eval { (getprotobyname('ipv6-icmp'))[2] } ||
+      croak("Can't get ipv6-icmp protocol by name"); # 58
+    $self->{"pid"} = $$ & 0xffff;           # Save lower 16 bits of pid
+    $self->{"fh"} = FileHandle->new();
+    socket($self->{"fh"}, $AF_INET6, SOCK_RAW, $self->{"proto_num"}) ||
+      croak("icmp socket error - $!");
+    $self->_setopts();
+    if ($self->{'gateway'}) {
+      my $g = $self->{gateway};
+      my $ip = _resolv($g)
+        or croak("nonexistent gateway $g");
+      $self->{family} eq $AF_INET6
+        or croak("gateway requires the AF_INET6 family");
+      $ip->{family} eq $AF_INET6
+        or croak("gateway address needs to be IPv6");
+      my $IPV6_NEXTHOP = eval { Socket::IPV6_NEXTHOP() } || 48; # IPV6_3542NEXTHOP, or 21
+      setsockopt($self->{"fh"}, $IPPROTO_IPV6, $IPV6_NEXTHOP, _pack_sockaddr_in($ip))
+        or croak "error configuring gateway to $g NEXTHOP $!";
+    }
+    if (exists $self->{IPV6_USE_MIN_MTU}) {
+      my $IPV6_USE_MIN_MTU = eval { Socket::IPV6_USE_MIN_MTU() } || 42;
+      setsockopt($self->{"fh"}, $IPPROTO_IPV6, $IPV6_USE_MIN_MTU,
+                 pack("I*", $self->{'IPV6_USE_MIN_MT'}))
+        or croak "error configuring IPV6_USE_MIN_MT} $!";
+    }
+    if (exists $self->{IPV6_RECVPATHMTU}) {
+      my $IPV6_RECVPATHMTU = eval { Socket::IPV6_RECVPATHMTU() } || 43;
+      setsockopt($self->{"fh"}, $IPPROTO_IPV6, $IPV6_RECVPATHMTU,
+                 pack("I*", $self->{'RECVPATHMTU'}))
+        or croak "error configuring IPV6_RECVPATHMTU $!";
     }
     if ($self->{'tos'}) {
-      setsockopt($self->{"fh"}, IPPROTO_IP, IP_TOS, pack("I*", $self->{'tos'}))
+      my $proto = $self->{family} == AF_INET ? IPPROTO_IP : $IPPROTO_IPV6;
+      setsockopt($self->{"fh"}, $proto, IP_TOS, pack("I*", $self->{'tos'}))
         or croak "error configuring tos to $self->{'tos'} $!";
     }
     if ($self->{'ttl'}) {
-      setsockopt($self->{"fh"}, IPPROTO_IP, IP_TTL, pack("I*", $self->{'ttl'}))
+      my $proto = $self->{family} == AF_INET ? IPPROTO_IP : $IPPROTO_IPV6;
+      setsockopt($self->{"fh"}, $proto, IP_TTL, pack("I*", $self->{'ttl'}))
         or croak "error configuring ttl to $self->{'ttl'} $!";
     }
   }
@@ -178,8 +277,9 @@ sub new
   {
     $self->{"proto_num"} = eval { (getprotobyname('tcp'))[2] } ||
       croak("Can't get tcp protocol by name");
-    $self->{"port_num"} = (getservbyname('echo', 'tcp'))[2] ||
-      croak("Can't get tcp echo port by name");
+    $self->{"port_num"} = $self->{"port"}
+      || (getservbyname('echo', 'tcp'))[2]
+      ||  croak("Can't get tcp echo port by name");
     $self->{"fh"} = FileHandle->new();
   }
   elsif ($self->{"proto"} eq "syn")
@@ -202,40 +302,34 @@ sub new
     $self->{"syn"} = {};
     $self->{"stop_time"} = 0;
   }
-  elsif ($self->{"proto"} eq "external")
-  {
-    # No preliminary work needs to be done.
-  }
 
   return($self);
 }
 
 # Description: Set the local IP address from which pings will be sent.
-# For ICMP and UDP pings, this calls bind() on the already-opened socket;
-# for TCP pings, just saves the address to be used when the socket is
-# opened.  Returns non-zero if successful; croaks on error.
+# For ICMP, UDP and TCP pings, just saves the address to be used when 
+# the socket is opened.  Returns non-zero if successful; croaks on error.
 sub bind
 {
   my ($self,
       $local_addr         # Name or IP number of local interface
       ) = @_;
-  my ($ip                 # Packed IP number of $local_addr
+  my ($ip,                # Hash of addr (string), addr_in (packed), family
+      $h                 # resolved hash
       );
 
   croak("Usage: \$p->bind(\$local_addr)") unless @_ == 2;
   croak("already bound") if defined($self->{"local_addr"}) &&
     ($self->{"proto"} eq "udp" || $self->{"proto"} eq "icmp");
 
-  $ip = inet_aton($local_addr);
+  $ip = $self->_resolv($local_addr);
   croak("nonexistent local address $local_addr") unless defined($ip);
-  $self->{"local_addr"} = $ip; # Only used if proto is tcp
+  $self->{"local_addr"} = $ip;
 
-  if ($self->{"proto"} eq "udp" || $self->{"proto"} eq "icmp")
-  {
-  CORE::bind($self->{"fh"}, sockaddr_in(0, $ip)) ||
-    croak("$self->{'proto'} bind error - $!");
-  }
-  elsif (($self->{"proto"} ne "tcp") && ($self->{"proto"} ne "syn"))
+  if (($self->{"proto"} ne "udp") && 
+      ($self->{"proto"} ne "icmp") && 
+      ($self->{"proto"} ne "tcp") && 
+      ($self->{"proto"} ne "syn"))
   {
     croak("Unknown protocol \"$self->{proto}\" in bind()");
   }
@@ -310,6 +404,94 @@ sub retrans
   $self->{"retrans"} = shift;
 }
 
+sub _IsAdminUser {
+  return unless $^O eq 'MSWin32' or $^O eq "cygwin";
+  return unless eval { require Win32 };
+  return unless defined &Win32::IsAdminUser;
+  return Win32::IsAdminUser();
+}
+
+sub _isroot {
+  if (($> and $^O ne 'VMS' and $^O ne 'cygwin')
+    or (($^O eq 'MSWin32' or $^O eq 'cygwin')
+        and !_IsAdminUser())
+    or ($^O eq 'VMS'
+        and (`write sys\$output f\$privilege("SYSPRV")` =~ m/FALSE/))) {
+      return 0;
+  }
+  else {
+    return 1;
+  }
+}
+
+# Description: Sets ipv6 reachability
+# REACHCONF was removed in RFC3542, ping6 -R supports it. requires root.
+
+sub IPV6_REACHCONF
+{
+  my $self = shift;
+  my $on = shift;
+  if ($on) {
+    my $reachconf = eval { Socket::IPV6_REACHCONF() };
+    if (!$reachconf) {
+      carp "IPV6_REACHCONF not supported on this platform";
+      return 0;
+    }
+    if (!_isroot()) {
+      carp "IPV6_REACHCONF requires root permissions";
+      return 0;
+    }
+    $self->{"IPV6_REACHCONF"} = 1;
+  }
+  else {
+    return $self->{"IPV6_REACHCONF"};
+  }
+}
+
+# Description: set it on or off.
+
+sub IPV6_USE_MIN_MTU
+{
+  my $self = shift;
+  my $on = shift;
+  if (defined $on) {
+    my $IPV6_USE_MIN_MTU = eval { Socket::IPV6_USE_MIN_MTU() } || 43;
+    #if (!$IPV6_USE_MIN_MTU) {
+    #  carp "IPV6_USE_MIN_MTU not supported on this platform";
+    #  return 0;
+    #}
+    $self->{"IPV6_USE_MIN_MTU"} = $on ? 1 : 0;
+    setsockopt($self->{"fh"}, $IPPROTO_IPV6, $IPV6_USE_MIN_MTU,
+               pack("I*", $self->{'IPV6_USE_MIN_MT'}))
+      or croak "error configuring IPV6_USE_MIN_MT} $!";
+  }
+  else {
+    return $self->{"IPV6_USE_MIN_MTU"};
+  }
+}
+
+# Description: notify an according MTU
+
+sub IPV6_RECVPATHMTU
+{
+  my $self = shift;
+  my $on = shift;
+  if ($on) {
+    my $IPV6_RECVPATHMTU = eval { Socket::IPV6_RECVPATHMTU() } || 43;
+    #if (!$RECVPATHMTU) {
+    #  carp "IPV6_RECVPATHMTU not supported on this platform";
+    #  return 0;
+    #}
+    $self->{"IPV6_RECVPATHMTU"} = 1;
+    setsockopt($self->{"fh"}, $IPPROTO_IPV6, $IPV6_RECVPATHMTU,
+               pack("I*", $self->{'IPV6_RECVPATHMTU'}))
+      or croak "error configuring IPV6_RECVPATHMTU} $!";
+  }
+  else {
+    return $self->{"IPV6_RECVPATHMTU"};
+  }
+}
+
 # Description: allows the module to use milliseconds as returned by
 # the Time::HiRes module
 
@@ -364,17 +546,33 @@ sub ping
   my ($self,
       $host,              # Name or IP number of host to ping
       $timeout,           # Seconds after which ping times out
+      $family,            # Address family
       ) = @_;
-  my ($ip,                # Packed IP number of $host
+  my ($ip,                # Hash of addr (string), addr_in (packed), family
       $ret,               # The return value
       $ping_time,         # When ping began
       );
 
-  croak("Usage: \$p->ping(\$host [, \$timeout])") unless @_ == 2 || @_ == 3;
+  $host = $self->{host} if !defined $host and $self->{host};
+  croak("Usage: \$p->ping([ \$host [, \$timeout [, \$family]]])") if @_ > 4 or !$host;
   $timeout = $self->{"timeout"} unless $timeout;
   croak("Timeout must be greater than 0 seconds") if $timeout <= 0;
 
-  $ip = inet_aton($host);
+  if ($family) {
+    if ($family =~ $qr_family) {
+      if ($family =~ $qr_family4) {
+        $self->{"family_local"} = AF_INET;
+      } else {
+        $self->{"family_local"} = $AF_INET6;
+      }
+    } else {
+      croak('Family must be "ipv4" or "ipv6"')
+    }
+  } else {
+    $self->{"family_local"} = $self->{"family"};
+  }
+  
+  $ip = $self->_resolv($host);
   return () unless defined($ip);      # Does host exist?
 
   # Dispatch to the appropriate routine.
@@ -388,6 +586,9 @@ sub ping
   elsif ($self->{"proto"} eq "icmp") {
     $ret = $self->ping_icmp($ip, $timeout);
   }
+  elsif ($self->{"proto"} eq "icmpv6") {
+    $ret = $self->ping_icmpv6($ip, $timeout);
+  }
   elsif ($self->{"proto"} eq "tcp") {
     $ret = $self->ping_tcp($ip, $timeout);
   }
@@ -406,33 +607,41 @@ sub ping
 # Uses Net::Ping::External to do an external ping.
 sub ping_external {
   my ($self,
-      $ip,                # Packed IP number of the host
-      $timeout            # Seconds after which ping times out
+      $ip,                # Hash of addr (string), addr_in (packed), family
+      $timeout,           # Seconds after which ping times out
+      $family
      ) = @_;
 
-  eval {
-    local @INC = @INC;
-    pop @INC if $INC[-1] eq '.';
-    require Net::Ping::External;
-  }
+  $ip = $self->{host} if !defined $ip and $self->{host};
+  $timeout = $self->{timeout} if !defined $timeout and $self->{timeout};
+
+  eval { require Net::Ping::External; }
     or croak('Protocol "external" not supported on your system: Net::Ping::External not found');
-  return Net::Ping::External::ping(ip => $ip, timeout => $timeout);
+  return Net::Ping::External::ping(ip => $ip->{host}, timeout => $timeout,
+                                   family => $family);
 }
 
+# h2ph "asm/socket.h"
+# require "asm/socket.ph";
+use constant SO_BINDTODEVICE  => 25;
 use constant ICMP_ECHOREPLY   => 0; # ICMP packet types
+use constant ICMPv6_ECHOREPLY => 129; # ICMP packet types
 use constant ICMP_UNREACHABLE => 3; # ICMP packet types
+use constant ICMPv6_UNREACHABLE => 1; # ICMP packet types
 use constant ICMP_ECHO        => 8;
+use constant ICMPv6_ECHO      => 128;
 use constant ICMP_TIME_EXCEEDED => 11; # ICMP packet types
 use constant ICMP_PARAMETER_PROBLEM => 12; # ICMP packet types
 use constant ICMP_STRUCT      => "C2 n3 A"; # Structure of a minimal ICMP packet
 use constant SUBCODE          => 0; # No ICMP subcode for ECHO and ECHOREPLY
 use constant ICMP_FLAGS       => 0; # No special flags for send or recv
 use constant ICMP_PORT        => 0; # No port with ICMP
+use constant IP_MTU_DISCOVER  => 10; # linux only
 
 sub ping_icmp
 {
   my ($self,
-      $ip,                # Packed IP number of the host
+      $ip,                # Hash of addr (string), addr_in (packed), family
       $timeout            # Seconds after which ping times out
       ) = @_;
 
@@ -457,15 +666,40 @@ sub ping_icmp
       $from_msg           # ICMP message
       );
 
+  $ip = $self->{host} if !defined $ip and $self->{host};
+  $timeout = $self->{timeout} if !defined $timeout and $self->{timeout};
+
+  socket($self->{"fh"}, $ip->{"family"}, SOCK_RAW, $self->{"proto_num"}) ||
+    croak("icmp socket error - $!");
+
+  if (defined $self->{"local_addr"} &&
+      !CORE::bind($self->{"fh"}, _pack_sockaddr_in(0, $self->{"local_addr"}))) {
+    croak("icmp bind error - $!");
+  }
+  $self->_setopts();
+
   $self->{"seq"} = ($self->{"seq"} + 1) % 65536; # Increment sequence
   $checksum = 0;                          # No checksum for starters
-  $msg = pack(ICMP_STRUCT . $self->{"data_size"}, ICMP_ECHO, SUBCODE,
-              $checksum, $self->{"pid"}, $self->{"seq"}, $self->{"data"});
+  if ($ip->{"family"} == AF_INET) {
+    $msg = pack(ICMP_STRUCT . $self->{"data_size"}, ICMP_ECHO, SUBCODE,
+                $checksum, $self->{"pid"}, $self->{"seq"}, $self->{"data"});
+  } else {
+                                          # how to get SRC
+    my $pseudo_header = pack('a16a16Nnn', $ip->{"addr_in"}, $ip->{"addr_in"}, 8+length($self->{"data"}), "\0", 0x003a);
+    $msg = pack(ICMP_STRUCT . $self->{"data_size"}, ICMPv6_ECHO, SUBCODE,
+                $checksum, $self->{"pid"}, $self->{"seq"}, $self->{"data"});
+    $msg = $pseudo_header.$msg
+  }
   $checksum = Net::Ping->checksum($msg);
-  $msg = pack(ICMP_STRUCT . $self->{"data_size"}, ICMP_ECHO, SUBCODE,
-              $checksum, $self->{"pid"}, $self->{"seq"}, $self->{"data"});
+  if ($ip->{"family"} == AF_INET) {
+    $msg = pack(ICMP_STRUCT . $self->{"data_size"}, ICMP_ECHO, SUBCODE,
+                $checksum, $self->{"pid"}, $self->{"seq"}, $self->{"data"});
+  } else {
+    $msg = pack(ICMP_STRUCT . $self->{"data_size"}, ICMPv6_ECHO, SUBCODE,
+                $checksum, $self->{"pid"}, $self->{"seq"}, $self->{"data"});
+  }
   $len_msg = length($msg);
-  $saddr = sockaddr_in(ICMP_PORT, $ip);
+  $saddr = _pack_sockaddr_in(ICMP_PORT, $ip);
   $self->{"from_ip"} = undef;
   $self->{"from_type"} = undef;
   $self->{"from_subcode"} = undef;
@@ -491,11 +725,14 @@ sub ping_icmp
       $from_pid = -1;
       $from_seq = -1;
       $from_saddr = recv($self->{"fh"}, $recv_msg, 1500, ICMP_FLAGS);
-      ($from_port, $from_ip) = sockaddr_in($from_saddr);
+      ($from_port, $from_ip) = _unpack_sockaddr_in($from_saddr, $ip->{"family"});
       ($from_type, $from_subcode) = unpack("C2", substr($recv_msg, 20, 2));
       if ($from_type == ICMP_ECHOREPLY) {
         ($from_pid, $from_seq) = unpack("n3", substr($recv_msg, 24, 4))
           if length $recv_msg >= 28;
+      } elsif ($from_type == ICMPv6_ECHOREPLY) {
+        ($from_pid, $from_seq) = unpack("n3", substr($recv_msg, 24, 4))
+          if length $recv_msg >= 28;
       } else {
         ($from_pid, $from_seq) = unpack("n3", substr($recv_msg, 52, 4))
           if length $recv_msg >= 56;
@@ -506,10 +743,10 @@ sub ping_icmp
       next if ($from_pid != $self->{"pid"});
       next if ($from_seq != $self->{"seq"});
       if (! $source_verify || ($self->ntop($from_ip) eq $self->ntop($ip))) { # Does the packet check out?
-        if ($from_type == ICMP_ECHOREPLY) {
+        if (($from_type == ICMP_ECHOREPLY) || ($from_type == ICMPv6_ECHOREPLY)) {
           $ret = 1;
-               $done = 1;
-        } elsif ($from_type == ICMP_UNREACHABLE) {
+          $done = 1;
+        } elsif (($from_type == ICMP_UNREACHABLE) || ($from_type == ICMPv6_UNREACHABLE)) {
           $done = 1;
         } elsif ($from_type == ICMP_TIME_EXCEEDED) {
           $ret = 0;
@@ -523,11 +760,16 @@ sub ping_icmp
   return $ret;
 }
 
+sub ping_icmpv6
+{
+  shift->ping_icmp(@_);
+}
+
 sub icmp_result {
   my ($self) = @_;
-  my $ip = $self->{"from_ip"} || "";
-  $ip = "\0\0\0\0" unless 4 == length $ip;
-  return ($self->ntop($ip),($self->{"from_type"} || 0), ($self->{"from_subcode"} || 0));
+  my $addr = $self->{"from_ip"} || "";
+  $addr = "\0\0\0\0" unless 4 == length $addr;
+  return ($self->ntop($addr),($self->{"from_type"} || 0), ($self->{"from_subcode"} || 0));
 }
 
 # Description:  Do a checksum on the message.  Basically sum all of
@@ -570,12 +812,15 @@ sub checksum
 sub ping_tcp
 {
   my ($self,
-      $ip,                # Packed IP number of the host
+      $ip,                # Hash of addr (string), addr_in (packed), family
       $timeout            # Seconds after which ping times out
       ) = @_;
   my ($ret                # The return value
       );
 
+  $ip = $self->{host} if !defined $ip and $self->{host};
+  $timeout = $self->{timeout} if !defined $timeout and $self->{timeout};
+
   $! = 0;
   $ret = $self -> tcp_connect( $ip, $timeout);
   if (!$self->{"econnrefused"} &&
@@ -589,33 +834,29 @@ sub ping_tcp
 sub tcp_connect
 {
   my ($self,
-      $ip,                # Packed IP number of the host
+      $ip,                # Hash of addr (string), addr_in (packed), family
       $timeout            # Seconds after which connect times out
       ) = @_;
   my ($saddr);            # Packed IP and Port
 
-  $saddr = sockaddr_in($self->{"port_num"}, $ip);
+  $ip = $self->{host} if !defined $ip and $self->{host};
+  $timeout = $self->{timeout} if !defined $timeout and $self->{timeout};
+
+  $saddr = _pack_sockaddr_in($self->{"port_num"}, $ip);
 
   my $ret = 0;            # Default to unreachable
 
   my $do_socket = sub {
-    socket($self->{"fh"}, PF_INET, SOCK_STREAM, $self->{"proto_num"}) ||
+    socket($self->{"fh"}, $ip->{"family"}, SOCK_STREAM, $self->{"proto_num"}) ||
       croak("tcp socket error - $!");
     if (defined $self->{"local_addr"} &&
-        !CORE::bind($self->{"fh"}, sockaddr_in(0, $self->{"local_addr"}))) {
+        !CORE::bind($self->{"fh"}, _pack_sockaddr_in(0, $self->{"local_addr"}))) {
       croak("tcp bind error - $!");
     }
-    if ($self->{'device'}) {
-      setsockopt($self->{"fh"}, SOL_SOCKET, SO_BINDTODEVICE(), pack("Z*", $self->{'device'}))
-        or croak("error binding to device $self->{'device'} $!");
-    }
-    if ($self->{'tos'}) {
-      setsockopt($self->{"fh"}, IPPROTO_IP, IP_TOS, pack("I*", $self->{'tos'}))
-        or croak "error configuring tos to $self->{'tos'} $!";
-    }
+    $self->_setopts();
   };
   my $do_connect = sub {
-    $self->{"ip"} = $ip;
+    $self->{"ip"} = $ip->{"addr_in"};
     # ECONNREFUSED is 10061 on MSWin32. If we pass it as child error through $?,
     # we'll get (10061 & 255) = 77, so we cannot check it in the parent process.
     return ($ret = connect($self->{"fh"}, $saddr) || ($! == ECONNREFUSED && !$self->{"econnrefused"}));
@@ -691,7 +932,7 @@ sub tcp_connect
 
     # Unset O_NONBLOCK property on filehandle
     $self->socket_blocking_mode($self->{"fh"}, 1);
-    $self->{"ip"} = $ip;
+    $self->{"ip"} = $ip->{"addr_in"};
     return $ret;
   };
 
@@ -784,9 +1025,10 @@ sub DESTROY {
 # back.  It returns 1 on success, 0 on failure.
 sub tcp_echo
 {
-  my $self = shift;
-  my $timeout = shift;
-  my $pingstring = shift;
+  my ($self, $timeout, $pingstring) = @_;
+
+  $timeout = $self->{timeout} if !defined $timeout and $self->{timeout};
+  $pingstring = $self->{pingstring} if !defined $pingstring and $self->{pingstring};
 
   my $ret = undef;
   my $time = &time();
@@ -835,9 +1077,6 @@ EOM
   return $ret;
 }
 
-
-
-
 # Description: Perform a stream ping.  If the tcp connection isn't
 # already open, it opens it.  It then sends some data and waits for
 # a reply.  It leaves the stream open on exit.
@@ -845,7 +1084,7 @@ EOM
 sub ping_stream
 {
   my ($self,
-      $ip,                # Packed IP number of the host
+      $ip,                # Hash of addr (string), addr_in (packed), family
       $timeout            # Seconds after which ping times out
       ) = @_;
 
@@ -855,7 +1094,7 @@ sub ping_stream
   }
 
   croak "tried to switch servers while stream pinging"
-    if $self->{"ip"} ne $ip;
+    if $self->{"ip"} ne $ip->{"addr_in"};
 
   return $self->tcp_echo($timeout, $pingstring);
 }
@@ -867,11 +1106,27 @@ sub open
 {
   my ($self,
       $host,              # Host or IP address
-      $timeout            # Seconds after which open times out
+      $timeout,           # Seconds after which open times out
+      $family
       ) = @_;
+  my $ip;                 # Hash of addr (string), addr_in (packed), family
+  $host = $self->{host} unless defined $host;
+
+  if ($family) {
+    if ($family =~ $qr_family) {
+      if ($family =~ $qr_family4) {
+        $self->{"family_local"} = AF_INET;
+      } else {
+        $self->{"family_local"} = $AF_INET6;
+      }
+    } else {
+      croak('Family must be "ipv4" or "ipv6"')
+    }
+  } else {
+    $self->{"family_local"} = $self->{"family"};
+  }
 
-  my ($ip);               # Packed IP number of the host
-  $ip = inet_aton($host);
+  $ip = $self->_resolv($host);
   $timeout = $self->{"timeout"} unless $timeout;
 
   if($self->{"proto"} eq "stream") {
@@ -883,6 +1138,43 @@ sub open
   }
 }
 
+sub _dontfrag {
+  my $self = shift;
+  # bsd solaris
+  my $IP_DONTFRAG = eval { Socket::IP_DONTFRAG() };
+  if ($IP_DONTFRAG) {
+    my $i = 1;
+    setsockopt($self->{"fh"}, IPPROTO_IP, $IP_DONTFRAG, pack("I*", $i))
+      or croak "error configuring IP_DONTFRAG $!";
+    # Linux needs more: Path MTU Discovery as defined in RFC 1191
+    # For non SOCK_STREAM sockets it is the user's responsibility to packetize
+    # the data in MTU sized chunks and to do the retransmits if necessary.
+    # The kernel will reject packets that are bigger than the known path
+    # MTU if this flag is set (with EMSGSIZE).
+    if ($^O eq 'linux') {
+      my $i = 2; # IP_PMTUDISC_DO
+      setsockopt($self->{"fh"}, IPPROTO_IP, IP_MTU_DISCOVER, pack("I*", $i))
+        or croak "error configuring IP_MTU_DISCOVER $!";
+    }
+  }
+}
+
+# SO_BINDTODEVICE + IP_TOS
+sub _setopts {
+  my $self = shift;
+  if ($self->{'device'}) {
+    setsockopt($self->{"fh"}, SOL_SOCKET, SO_BINDTODEVICE, pack("Z*", $self->{'device'}))
+      or croak "error binding to device $self->{'device'} $!";
+  }
+  if ($self->{'tos'}) { # need to re-apply ToS (RT #6706)
+    setsockopt($self->{"fh"}, IPPROTO_IP, IP_TOS, pack("I*", $self->{'tos'}))
+      or croak "error applying tos to $self->{'tos'} $!";
+  }
+  if ($self->{'dontfrag'}) {
+    $self->_dontfrag;
+  }
+}  
+
 
 # Description:  Perform a udp echo ping.  Construct a message of
 # at least the one-byte sequence number and any additional data bytes.
@@ -895,7 +1187,7 @@ use constant UDP_FLAGS => 0; # Nothing special on send or recv
 sub ping_udp
 {
   my ($self,
-      $ip,                # Packed IP number of the host
+      $ip,                # Hash of addr (string), addr_in (packed), family
       $timeout            # Seconds after which ping times out
       ) = @_;
 
@@ -914,10 +1206,21 @@ sub ping_udp
       $from_ip            # Packed IP number of sender
       );
 
-  $saddr = sockaddr_in($self->{"port_num"}, $ip);
+  $saddr = _pack_sockaddr_in($self->{"port_num"}, $ip);
   $self->{"seq"} = ($self->{"seq"} + 1) % 256;    # Increment sequence
   $msg = chr($self->{"seq"}) . $self->{"data"};   # Add data if any
 
+  socket($self->{"fh"}, $ip->{"family"}, SOCK_DGRAM,
+         $self->{"proto_num"}) ||
+           croak("udp socket error - $!");
+
+  if (defined $self->{"local_addr"} &&
+      !CORE::bind($self->{"fh"}, _pack_sockaddr_in(0, $self->{"local_addr"}))) {
+    croak("udp bind error - $!");
+  }
+
+  $self->_setopts();
+
   if ($self->{"connected"}) {
     if ($self->{"connected"} ne $saddr) {
       # Still connected to wrong destination.
@@ -938,8 +1241,9 @@ sub ping_udp
   if ($flush) {
     # Need to socket() again to flush the descriptor
     # This will disconnect from the old saddr.
-    socket($self->{"fh"}, PF_INET, SOCK_DGRAM,
+    socket($self->{"fh"}, $ip->{"family"}, SOCK_DGRAM,
            $self->{"proto_num"});
+    $self->_setopts();
   }
   # Connect the socket if it isn't already connected
   # to the right destination.
@@ -987,7 +1291,7 @@ sub ping_udp
         }
         $done = 1;
       } else {
-        ($from_port, $from_ip) = sockaddr_in($from_saddr);
+        ($from_port, $from_ip) = _unpack_sockaddr_in($from_saddr, $ip->{"family"});
         if (!$source_verify ||
             (($from_ip eq $ip) &&        # Does the packet check out?
              ($from_port == $self->{"port_num"}) &&
@@ -1037,26 +1341,19 @@ sub ping_syn
   }
 
   my $fh = FileHandle->new();
-  my $saddr = sockaddr_in($self->{"port_num"}, $ip);
+  my $saddr = _pack_sockaddr_in($self->{"port_num"}, $ip);
 
   # Create TCP socket
-  if (!socket ($fh, PF_INET, SOCK_STREAM, $self->{"proto_num"})) {
+  if (!socket ($fh, $ip->{"family"}, SOCK_STREAM, $self->{"proto_num"})) {
     croak("tcp socket error - $!");
   }
 
   if (defined $self->{"local_addr"} &&
-      !CORE::bind($fh, sockaddr_in(0, $self->{"local_addr"}))) {
+      !CORE::bind($fh, _pack_sockaddr_in(0, $self->{"local_addr"}))) {
     croak("tcp bind error - $!");
   }
 
-  if ($self->{'device'}) {
-    setsockopt($fh, SOL_SOCKET, SO_BINDTODEVICE(), pack("Z*", $self->{'device'}))
-      or croak("error binding to device $self->{'device'} $!");
-  }
-  if ($self->{'tos'}) {
-    setsockopt($fh, IPPROTO_IP, IP_TOS, pack("I*", $self->{'tos'}))
-      or croak "error configuring tos to $self->{'tos'} $!";
-  }
+  $self->_setopts();
   # Set O_NONBLOCK property on filehandle
   $self->socket_blocking_mode($fh, 0);
 
@@ -1106,26 +1403,19 @@ sub ping_syn_fork {
       }
     } else {
       # Child process
-      my $saddr = sockaddr_in($self->{"port_num"}, $ip);
+      my $saddr = _pack_sockaddr_in($self->{"port_num"}, $ip);
 
       # Create TCP socket
-      if (!socket ($self->{"fh"}, PF_INET, SOCK_STREAM, $self->{"proto_num"})) {
+      if (!socket ($self->{"fh"}, $ip->{"family"}, SOCK_STREAM, $self->{"proto_num"})) {
         croak("tcp socket error - $!");
       }
 
       if (defined $self->{"local_addr"} &&
-          !CORE::bind($self->{"fh"}, sockaddr_in(0, $self->{"local_addr"}))) {
+          !CORE::bind($self->{"fh"}, _pack_sockaddr_in(0, $self->{"local_addr"}))) {
         croak("tcp bind error - $!");
       }
 
-      if ($self->{'device'}) {
-        setsockopt($self->{"fh"}, SOL_SOCKET, SO_BINDTODEVICE(), pack("Z*", $self->{'device'}))
-          or croak("error binding to device $self->{'device'} $!");
-      }
-      if ($self->{'tos'}) {
-        setsockopt($self->{"fh"}, IPPROTO_IP, IP_TOS, pack("I*", $self->{'tos'}))
-          or croak "error configuring tos to $self->{'tos'} $!";
-      }
+      $self->_setopts();
 
       $!=0;
       # Try to connect (could take a long time)
@@ -1159,8 +1449,9 @@ sub ack
     }
     my $wbits = "";
     my $stop_time = 0;
-    if (my $host = shift) {
-      # Host passed as arg
+    if (my $host = shift or $self->{host}) {
+      # Host passed as arg or as option to new
+      $host = $self->{host} unless defined $host;
       if (exists $self->{"bad"}->{$host}) {
         if (!$self->{"econnrefused"} &&
             $self->{"bad"}->{ $host } &&
@@ -1417,7 +1708,7 @@ sub ntop {
     # Any port will work, even undef, but this will work for now.
     # Socket warns when undef is passed in, but it still works.
     my $port = getservbyname('echo', 'udp');
-    my $sockaddr = sockaddr_in $port, $ip;
+    my $sockaddr = _pack_sockaddr_in($port, $ip);
     my ($error, $address) = getnameinfo($sockaddr, NI_NUMERICHOST);
     if($error) {
       croak $error;
@@ -1425,6 +1716,208 @@ sub ntop {
     return $address;
 }
 
+sub wakeonlan {
+  my ($mac_addr, $host, $port) = @_;
+
+  # use the discard service if $port not passed in
+  if (! defined $host) { $host = '255.255.255.255' }
+  if (! defined $port || $port !~ /^\d+$/ ) { $port = 9 }
+
+  require IO::Socket::INET;
+  my $sock = IO::Socket::INET->new(Proto=>'udp') || return undef;
+
+  my $ip_addr = inet_aton($host);
+  my $sock_addr = sockaddr_in($port, $ip_addr);
+  $mac_addr =~ s/://g;
+  my $packet = pack('C6H*', 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, $mac_addr x 16);
+
+  setsockopt($sock, SOL_SOCKET, SO_BROADCAST, 1);
+  send($sock, $packet, 0, $sock_addr);
+  $sock->close;
+
+  return 1;
+}
+
+########################################################
+# DNS hostname resolution
+# return:
+#   $h->{name}    = host - as passed in
+#   $h->{host}    = host - as passed in without :port
+#   $h->{port}    = OPTIONAL - if :port, then value of port
+#   $h->{addr}    = resolved numeric address
+#   $h->{addr_in} = aton/pton result
+#   $h->{family}  = AF_INET/6
+############################
+sub _resolv {
+  my ($self,
+      $name,
+      ) = @_;
+
+  my %h;
+  $h{name} = $name;
+  my $family = $self->{"family"};
+
+  if (defined($self->{"family_local"})) {
+    $family = $self->{"family_local"}
+  }
+
+# START - host:port
+  my $cnt = 0;
+
+  # Count ":"
+  $cnt++ while ($name =~ m/:/g);
+
+  # 0 = hostname or IPv4 address
+  if ($cnt == 0) {
+    $h{host} = $name
+  # 1 = IPv4 address with port
+  } elsif ($cnt == 1) {
+    ($h{host}, $h{port}) = split /:/, $name
+  # >=2 = IPv6 address
+  } elsif ($cnt >= 2) {
+    #IPv6 with port - [2001::1]:port
+    if ($name =~ /^\[.*\]:\d{1,5}$/) {
+      ($h{host}, $h{port}) = split /:([^:]+)$/, $name # split after last :
+    # IPv6 without port
+    } else {
+      $h{host} = $name
+    }
+  }
+
+  # Clean up host
+  $h{host} =~ s/\[//g;
+  $h{host} =~ s/\]//g;
+  # Clean up port
+  if (defined($h{port}) && (($h{port} !~ /^\d{1,5}$/) || ($h{port} < 1) || ($h{port} > 65535))) {
+    croak("Invalid port `$h{port}' in `$name'");
+  }
+# END - host:port
+
+  # address check
+  # new way
+  if ($Socket::VERSION >= 1.94) {
+    my %hints = (
+      family   => $AF_UNSPEC,
+      protocol => IPPROTO_TCP,
+      flags => $AI_NUMERICHOST
+    );
+
+    # numeric address, return
+    my ($err, @getaddr) = Socket::getaddrinfo($h{host}, undef, \%hints);
+    if (defined($getaddr[0])) {
+      $h{addr} = $h{host};
+      $h{family} = $getaddr[0]->{family};
+      if ($h{family} == AF_INET) {
+        (undef, $h{addr_in}, undef, undef) = Socket::unpack_sockaddr_in $getaddr[0]->{addr};
+      } else {
+        (undef, $h{addr_in}, undef, undef) = Socket::unpack_sockaddr_in6 $getaddr[0]->{addr};
+      }
+      return \%h
+    }
+  # old way
+  } else {
+    # numeric address, return
+    my $ret = gethostbyname($h{host});
+    if (defined($ret) && (_inet_ntoa($ret) eq $h{host})) {
+      $h{addr} = $h{host};
+      $h{addr_in} = $ret;
+      $h{family} = AF_INET;
+      return \%h
+    }
+  }
+
+  # resolve
+  # new way
+  if ($Socket::VERSION >= 1.94) {
+    my %hints = (
+      family   => $family,
+      protocol => IPPROTO_TCP
+    );
+
+    my ($err, @getaddr) = Socket::getaddrinfo($h{host}, undef, \%hints);
+    if (defined($getaddr[0])) {
+      my ($err, $address) = Socket::getnameinfo($getaddr[0]->{addr}, $NI_NUMERICHOST);
+      if (defined($address)) {
+        $h{addr} = $address;
+        $h{addr} =~ s/\%(.)*$//; # remove %ifID if IPv6
+        $h{family} = $getaddr[0]->{family};
+        if ($h{family} == AF_INET) {
+          (undef, $h{addr_in}, undef, undef) = Socket::unpack_sockaddr_in $getaddr[0]->{addr};
+        } else {
+          (undef, $h{addr_in}, undef, undef) = Socket::unpack_sockaddr_in6 $getaddr[0]->{addr};
+        }
+        return \%h
+      } else {
+        croak("getnameinfo($getaddr[0]->{addr}) failed - $err");
+      }
+    } else {
+      my $error = sprintf "getaddrinfo($h{host},,%s) failed - $err",
+                  ($family == AF_INET) ? "AF_INET" : "AF_INET6";
+      croak("$error");
+    }
+  # old way
+  } else {
+    if ($family == $AF_INET6) {
+      croak("Socket >= 1.94 required for IPv6 - found Socket $Socket::VERSION");
+    }
+
+    my @gethost = gethostbyname($h{host});
+    if (defined($gethost[4])) {
+      $h{addr} = inet_ntoa($gethost[4]);
+      $h{addr_in} = $gethost[4];
+      $h{family} = AF_INET;
+      return \%h
+    } else {
+      croak("gethostbyname($h{host}) failed - $^E");
+    }
+  }
+}
+
+sub _pack_sockaddr_in($$) {
+  my ($port,
+      $addr,
+      ) = @_;
+
+  if ($addr->{"family"} == AF_INET) {
+    return Socket::pack_sockaddr_in($port, $addr->{"addr_in"});
+  } else {
+    return Socket::pack_sockaddr_in6($port, $addr->{"addr_in"});
+  }
+}
+
+sub _unpack_sockaddr_in($;$) {
+  my ($addr,
+      $family,
+      ) = @_;
+
+  my ($port, $host);
+  if ($family == AF_INET || (!defined($family) and length($addr) <= 16 )) {
+    ($port, $host) = Socket::unpack_sockaddr_in($addr);
+  } else {
+    ($port, $host) = Socket::unpack_sockaddr_in6($addr);
+  }
+  return $port, $host
+}
+
+sub _inet_ntoa {
+  my ($addr
+      ) = @_;
+
+  my $ret;
+  if ($Socket::VERSION >= 1.94) {
+    my ($err, $address) = Socket::getnameinfo($addr, $NI_NUMERICHOST);
+    if (defined($address)) {
+      $ret = $address;
+    } else {
+      croak("getnameinfo($addr) failed - $err");
+    }
+  } else {
+    $ret = inet_ntoa($addr)
+  }
+    
+  return $ret
+}
+
 1;
 __END__
 
@@ -1546,33 +2039,69 @@ This protocol does not require any special privileges.
 
 =over 4
 
-=item Net::Ping->new([$proto [, $def_timeout [, $bytes [, $device [, $tos [, $ttl ]]]]]]);
+=item Net::Ping->new([proto, timeout, bytes, device, tos, ttl, family,
+                      host, port, bind, gateway, retrans, pingstring,
+                      source_verify econnrefused dontfrag
+                      IPV6_USE_MIN_MTU IPV6_RECVPATHMTU])
 
-Create a new ping object.  All of the parameters are optional.  $proto
-specifies the protocol to use when doing a ping.  The current choices
-are "tcp", "udp", "icmp", "stream", "syn", or "external".
-The default is "tcp".
+Create a new ping object.  All of the parameters are optional and can
+be passed as hash ref.  All options besides the first 7 must be passed
+as hash ref.
 
-If a default timeout ($def_timeout) in seconds is provided, it is used
+C<proto> specifies the protocol to use when doing a ping.  The current
+choices are "tcp", "udp", "icmp", "icmpv6", "stream", "syn", or
+"external".  The default is "tcp".
+
+If a C<timeout> in seconds is provided, it is used
 when a timeout is not given to the ping() method (below).  The timeout
 must be greater than 0 and the default, if not specified, is 5 seconds.
 
-If the number of data bytes ($bytes) is given, that many data bytes
+If the number of data bytes (C<bytes>) is given, that many data bytes
 are included in the ping packet sent to the remote host. The number of
 data bytes is ignored if the protocol is "tcp".  The minimum (and
 default) number of data bytes is 1 if the protocol is "udp" and 0
 otherwise.  The maximum number of data bytes that can be specified is
 1024.
 
-If $device is given, this device is used to bind the source endpoint
+If C<device> is given, this device is used to bind the source endpoint
 before sending the ping packet.  I believe this only works with
 superuser privileges and with udp and icmp protocols at this time.
 
-If $tos is given, this ToS is configured into the socket.
+If <tos> is given, this ToS is configured into the socket.
+
+For icmp, C<ttl> can be specified to set the TTL of the outgoing packet.
+
+Valid C<family> values for IPv4:
+
+   4, v4, ip4, ipv4, AF_INET (constant)
+
+Valid C<family> values for IPv6:
+
+   6, v6, ip6, ipv6, AF_INET6 (constant)
+
+The C<host> argument implicitly specifies the family if the family
+argument is not given.
+
+The C<port> argument is only valid for a udp, tcp or stream ping, and will not
+do what you think it does. ping returns true when we get a "Connection refused"!
+The default is the echo port.
+
+The C<bind> argument specifies the local_addr to bind to.
+By specifying a bind argument you don't need the bind method.
+
+The C<gateway> argument is only valid for IPv6, and requires a IPv6
+address.
 
-For icmp, $ttl can be specified to set the TTL of the outgoing packet.
+The C<retrans> argument the exponential backoff rate, default 1.2.
+It matches the $def_factor global.
 
-=item $p->ping($host [, $timeout]);
+The C<dontfrag> argument sets the IP_DONTFRAG bit, but note that
+IP_DONTFRAG is not yet defined by Socket, and not available on many
+systems. Then it is ignored. On linux it also sets IP_MTU_DISCOVER to
+IP_PMTUDISC_DO but need we don't chunk oversized packets. You need to
+set $data_size manually.
+
+=item $p->ping($host [, $timeout [, $family]]);
 
 Ping the remote host and wait for a response.  $host can be either the
 hostname or the IP number of the remote host.  The optional timeout
@@ -1627,10 +2156,44 @@ Deprecated method, but does the same as service_check() method.
 
 =item $p->hires( { 0 | 1 } );
 
-Causes this module to use Time::HiRes module, allowing milliseconds
+With 1 causes this module to use Time::HiRes module, allowing milliseconds
 to be returned by subsequent calls to ping().
 
-This is disabled by default.
+=item $p->time
+
+The current time, hires or not.
+
+=item $p->socket_blocking_mode( $fh, $mode );
+
+Sets or clears the O_NONBLOCK flag on a file handle.
+
+=item $p->IPV6_USE_MIN_MTU
+
+With argument sets the option.
+Without returns the option value.
+
+=item $p->IPV6_RECVPATHMTU
+
+Notify an according IPv6 MTU.
+
+With argument sets the option.
+Without returns the option value.
+
+=item $p->IPV6_HOPLIMIT
+
+With argument sets the option.
+Without returns the option value.
+
+=item $p->IPV6_REACHCONF I<NYI>
+
+Sets ipv6 reachability
+IPV6_REACHCONF was removed in RFC3542. ping6 -R supports it.
+IPV6_REACHCONF requires root/admin permissions.
+
+With argument sets the option.
+Without returns the option value.
+
+Not yet implemented.
 
 =item $p->bind($local_addr);
 
@@ -1646,6 +2209,9 @@ then bind() must be called at most once per object, and (if it is
 called at all) must be called before the first call to ping() for that
 object.
 
+The bind() call can be omitted when specifying the C<bind> option to
+new().
+
 =item $p->open($host);
 
 When you are using the "stream" protocol, this call pre-opens the
@@ -1657,6 +2223,9 @@ automatically opened the first time C<ping()> is called.
 This call simply does nothing if you are using any protocol other
 than stream.
 
+The $host argument can be omitted when specifying the C<host> option to
+new().
+
 =item $p->ack( [ $host ] );
 
 When using the "syn" protocol, use this method to determine
@@ -1676,12 +2245,75 @@ value will be pertaining to that host only.
 This call simply does nothing if you are using any protocol
 other than syn.
 
+When new() had a host option, this host will be used.
+Without host argument, all hosts are scanned.
+
 =item $p->nack( $failed_ack_host );
 
 The reason that host $failed_ack_host did not receive a
 valid ACK.  Useful to find out why when ack( $fail_ack_host )
 returns a false value.
 
+=item $p->ack_unfork($host)
+
+The variant called by ack() with the syn protocol and $syn_forking
+enabled.
+
+=item $p->ping_icmp([$host, $timeout, $family])
+
+The ping() method used with the icmp protocol.
+
+=item $p->ping_icmpv6([$host, $timeout, $family]) I<NYI>
+
+The ping() method used with the icmpv6 protocol.
+
+=item $p->ping_stream([$host, $timeout, $family])
+
+The ping() method used with the stream protocol.
+
+Perform a stream ping.  If the tcp connection isn't
+already open, it opens it.  It then sends some data and waits for
+a reply.  It leaves the stream open on exit.
+
+=item $p->ping_syn([$host, $ip, $start_time, $stop_time])
+
+The ping() method used with the syn protocol.
+Sends a TCP SYN packet to host specified.
+
+=item $p->ping_syn_fork([$host, $timeout, $family])
+
+The ping() method used with the forking syn protocol.
+
+=item $p->ping_tcp([$host, $timeout, $family])
+
+The ping() method used with the tcp protocol.
+
+=item $p->ping_udp([$host, $timeout, $family])
+
+The ping() method used with the udp protocol.
+
+Perform a udp echo ping.  Construct a message of
+at least the one-byte sequence number and any additional data bytes.
+Send the message out and wait for a message to come back.  If we
+get a message, make sure all of its parts match.  If they do, we are
+done.  Otherwise go back and wait for the message until we run out
+of time.  Return the result of our efforts.
+
+=item $p->ping_external([$host, $timeout, $family])
+
+The ping() method used with the external protocol.
+Uses Net::Ping::External to do an external ping.
+
+=item $p->tcp_connect([$ip, $timeout])
+
+Initiates a TCP connection, for a tcp ping.
+
+=item $p->tcp_echo([$ip, $timeout, $pingstring])
+
+Performs a TCP echo.
+It writes the given string to the socket and then reads it
+back.  It returns 1 on success, 0 on failure.
+
 =item $p->close();
 
 Close the network connection for this ping object.  The network
@@ -1697,6 +2329,24 @@ of calling C<$p-E<gt>service_check(1)> causing a ping to return a successful
 response only if that specific port is accessible.  This function returns
 the value of the port that C<ping()> will connect to.
 
+=item $p->mselect
+
+A select() wrapper that compensates for platform
+peculiarities.
+
+=item $p->ntop
+
+Platform abstraction over inet_ntop()
+
+=item $p->checksum($msg)
+
+Do a checksum on the message.  Basically sum all of
+the short words and fold the high order bits into the low order bits.
+
+=item $p->icmp_result
+
+Returns a list of addr, type, subcode.
+
 =item pingecho($host [, $timeout]);
 
 To provide backward compatibility with the previous version of
@@ -1706,6 +2356,17 @@ return values and parameters are the same as described for the ping()
 method.  This subroutine is obsolete and may be removed in a future
 version of Net::Ping.
 
+=item wakeonlan($mac, [$host, [$port]])
+
+Emit the popular wake-on-lan magic udp packet to wake up a local
+device.  See also L<Net::Wake>, but this has the mac address as 1st arg.
+$host should be the local gateway. Without it will broadcast.
+
+Default host: '255.255.255.255'
+Default port: 9
+
+  perl -MNet::Ping=wakeonlan -e'wakeonlan "e0:69:95:35:68:d2"'
+
 =back
 
 =head1 NOTES
@@ -1717,9 +2378,10 @@ either udp or icmp.  If many hosts are pinged frequently, you may wish
 to implement a small wait (e.g. 25ms or more) between each ping to
 avoid flooding your network with packets.
 
-The icmp protocol requires that the program be run as root or that it
-be setuid to root.  The other protocols do not require special
-privileges, but not all network devices implement tcp or udp echo.
+The icmp and icmpv6 protocols requires that the program be run as root
+or that it be setuid to root.  The other protocols do not require
+special privileges, but not all network devices implement tcp or udp
+echo.
 
 Local hosts should normally respond to pings within milliseconds.
 However, on a very congested network it may take up to 3 seconds or
@@ -1739,57 +2401,44 @@ kinds of ICMP packets.
 
 =head1 INSTALL
 
-The latest source tree is available via cvs:
+The latest source tree is available via git:
 
-  cvs -z3 -q -d \
-    :pserver:anonymous@cvs.roobik.com.:/usr/local/cvsroot/freeware \
-    checkout Net-Ping
+  git clone https://github.com/rurban/net-ping.git Net-Ping
   cd Net-Ping
 
 The tarball can be created as follows:
 
   perl Makefile.PL ; make ; make dist
 
-The latest Net::Ping release can be found at CPAN:
-
-  $CPAN/modules/by-module/Net/
-
-1) Extract the tarball
-
-  gtar -zxvf Net-Ping-xxxx.tar.gz
-  cd Net-Ping-xxxx
-
-2) Build:
+The latest Net::Ping releases are included in cperl and perl5.
 
-  make realclean
-  perl Makefile.PL
-  make
-  make test
-
-3) Install
-
-  make install
+=head1 BUGS
 
-Or install it RPM Style:
+For a list of known issues, visit:
 
-  rpm -ta SOURCES/Net-Ping-xxxx.tar.gz
+L<https://rt.cpan.org/NoAuth/Bugs.html?Dist=Net-Ping>
 
-  rpm -ih RPMS/noarch/perl-Net-Ping-xxxx.rpm
+To report a new bug, visit:
 
-=head1 BUGS
+L<https://rt.cpan.org/NoAuth/ReportBug.html?Queue=Net-Ping> (stale)
 
-For a list of known issues, visit:
+or call:
 
-https://rt.cpan.org/NoAuth/Bugs.html?Dist=Net-Ping
+  perlbug
 
-To report a new bug, visit:
+resp.:
 
-https://rt.cpan.org/NoAuth/ReportBug.html?Queue=Net-Ping
+  cperlbug
 
 =head1 AUTHORS
 
-  Current maintainer:
+  Current maintainers:
+    perl11 (for cperl, with IPv6 support and more)
+    p5p    (for perl5)
+
+  Previous maintainers:
     bbb@cpan.org (Rob Brown)
+    Steve Peters
 
   External protocol:
     colinm@cpan.org (Colin McMillen)
@@ -1797,6 +2446,9 @@ https://rt.cpan.org/NoAuth/ReportBug.html?Queue=Net-Ping
   Stream protocol:
     bronson@trestle.com (Scott Bronson)
 
+  Wake-on-lan:
+    1999-2003 Clinton Wong
+
   Original pingecho():
     karrer@bernina.ethz.ch (Andreas Karrer)
     pmarquess@bfsec.bt.co.uk (Paul Marquess)
@@ -1806,6 +2458,10 @@ https://rt.cpan.org/NoAuth/ReportBug.html?Queue=Net-Ping
 
 =head1 COPYRIGHT
 
+Copyright (c) 2016, cPanel Inc.  All rights reserved.
+
+Copyright (c) 2012, Steve Peters.  All rights reserved.
+
 Copyright (c) 2002-2003, Rob Brown.  All rights reserved.
 
 Copyright (c) 2001, Colin McMillen.  All rights reserved.
diff --git a/dist/Net-Ping/t/000_load.t b/dist/Net-Ping/t/000_load.t
new file mode 100644 (file)
index 0000000..3cb518d
--- /dev/null
@@ -0,0 +1,16 @@
+#!perl -T
+use 5.006;
+use strict;
+use warnings FATAL => 'all';
+use Test::More;
+
+plan tests => 3;
+
+BEGIN {
+    use_ok( 'Socket' )      || print "No Socket!\n";
+    use_ok( 'Time::HiRes' ) || print "No Time::HiRes!\n";
+    use_ok( 'Net::Ping' )   || print "No Net::Ping!\n";
+}
+
+diag( "Testing Net::Ping $Net::Ping::VERSION, Perl $], $^X" );
+
diff --git a/dist/Net-Ping/t/001_new.t b/dist/Net-Ping/t/001_new.t
new file mode 100644 (file)
index 0000000..d1c651d
--- /dev/null
@@ -0,0 +1,64 @@
+use warnings;
+use strict;
+
+use Test::More qw(no_plan);
+BEGIN {use_ok('Net::Ping')};
+
+# plain ol' constuctor call
+my $p = Net::Ping->new();
+isa_ok($p, "Net::Ping");
+
+# call new from an instantiated object
+my $p2 = $p->new();
+isa_ok($p2, "Net::Ping");
+
+# named args
+my $p3 = Net::Ping->new({proto => 'tcp', ttl => 5});
+isa_ok($p3, "Net::Ping");
+
+# check for invalid proto
+eval {
+    $p = Net::Ping->new("thwackkk");
+};
+like($@, qr/Protocol for ping must be "icmp", "icmpv6", "udp", "tcp", "syn", "stream" or "external"/, "new() errors for invalid protocol");
+
+# check for invalid timeout
+eval {
+    $p = Net::Ping->new("tcp", -1);
+};
+like($@, qr/Default timeout for ping must be greater than 0 seconds/, "new() errors for invalid timeout");
+
+# check for invalid data sizes
+eval {
+    $p = Net::Ping->new("udp", 10, -1);
+};
+like($@, qr/Data for ping must be from/, "new() errors for invalid data size");
+
+eval {
+    $p = Net::Ping->new("udp", 10, 1025);
+};
+like($@, qr/Data for ping must be from/, "new() errors for invalid data size");
+
+# force failures for udp
+
+
+# force failures for tcp
+SKIP: {
+    diag "Checking icmp";
+    eval { $p = Net::Ping->new('icmp'); };
+    if($> and $^O ne 'VMS' and $^O ne 'cygwin') {
+        like($@, qr/icmp ping requires root privilege/, "Need root for icmp");
+        skip "icmp tests require root", 2;
+    } else {
+        isa_ok($p, "Net::Ping");
+    }
+
+    # set IP TOS to "Minimum Delay"
+    $p = Net::Ping->new("icmp", undef, undef, undef, 8);
+    isa_ok($p, "Net::Ping");
+
+    # This really shouldn't work.  Not sure who to blame.
+    $p = Net::Ping->new("icmp", undef, undef, undef, "does this fail");
+    isa_ok($p, "Net::Ping");
+}
+
diff --git a/dist/Net-Ping/t/010_pingecho.t b/dist/Net-Ping/t/010_pingecho.t
new file mode 100644 (file)
index 0000000..c7d5786
--- /dev/null
@@ -0,0 +1,8 @@
+use warnings;
+use strict;
+
+use Test::More tests => 2;
+BEGIN {use_ok('Net::Ping')};
+
+my $result = pingecho("127.0.0.1");
+is($result, 1, "pingecho works");
diff --git a/dist/Net-Ping/t/100_load.t b/dist/Net-Ping/t/100_load.t
deleted file mode 100644 (file)
index fa04a0c..0000000
+++ /dev/null
@@ -1,12 +0,0 @@
-use strict;
-
-BEGIN {
-  unless (eval "require Socket") {
-    print "1..0 \# Skip: no Socket\n";
-    exit;
-  }
-}
-
-use Test::More tests => 1;
-# Just make sure everything compiles
-BEGIN {use_ok 'Net::Ping'};
index deddd8f..b7f0208 100644 (file)
@@ -20,18 +20,7 @@ BEGIN {use_ok('Net::Ping')};
 
 SKIP: {
   skip "icmp ping requires root privileges.", 1
-    if ($> and $^O ne 'VMS' and $^O ne 'cygwin')
-      or (($^O eq 'MSWin32' or $^O eq 'cygwin')
-         and !IsAdminUser())
-       or ($^O eq 'VMS'
-           and (`write sys\$output f\$privilege("SYSPRV")` =~ m/FALSE/));
+    unless &Net::Ping::_isroot;
   my $p = new Net::Ping "icmp";
   isa_ok($p, 'Net::Ping', 'object can be instantiated for icmp protocol');
 }
-
-sub IsAdminUser {
-  return unless $^O eq 'MSWin32' or $^O eq 'cygwin';
-  return unless eval { require Win32 };
-  return unless defined &Win32::IsAdminUser;
-  return Win32::IsAdminUser();
-}
index 62855ff..3391e6e 100644 (file)
@@ -20,18 +20,8 @@ BEGIN {use_ok('Net::Ping')};
 
 SKIP: {
   skip "icmp ping requires root privileges.", 1
-    if ($> and $^O ne 'VMS' and $^O ne 'cygwin')
-      or (($^O eq 'MSWin32' or $^O eq 'cygwin')
-         and !IsAdminUser())
-       or ($^O eq 'VMS'
-           and (`write sys\$output f\$privilege("SYSPRV")` =~ m/FALSE/));
+    if !Net::Ping::_isroot() or $^O eq 'MSWin32';
   my $p = new Net::Ping "icmp";
   is($p->ping("127.0.0.1"), 1, "icmp ping 127.0.0.1");
 }
 
-sub IsAdminUser {
-  return unless $^O eq 'MSWin32' or $^O eq "cygwin";
-  return unless eval { require Win32 };
-  return unless defined &Win32::IsAdminUser;
-  return Win32::IsAdminUser();
-}
index 75c8c49..d68793a 100644 (file)
@@ -19,11 +19,7 @@ BEGIN {use_ok('Net::Ping')};
 
 SKIP: {
   skip "icmp ping requires root privileges.", 1
-    if ($> and $^O ne 'VMS' and $^O ne 'cygwin')
-      or (($^O eq 'MSWin32' or $^O eq 'cygwin')
-    and !IsAdminUser())
-  or ($^O eq 'VMS'
-      and (`write sys\$output f\$privilege("SYSPRV")` =~ m/FALSE/));
+    if !Net::Ping::_isroot() or $^O eq 'MSWin32';
   my $p = new Net::Ping ("icmp",undef,undef,undef,undef,undef);
   isa_ok($p, 'Net::Ping');
   ok $p->ping("127.0.0.1");
@@ -44,10 +40,3 @@ SKIP: {
   ok $p->ping("127.0.0.1");
   $p->close();
 }
-
-sub IsAdminUser {
-  return unless $^O eq 'MSWin32' or $^O eq "cygwin";
-  return unless eval { require Win32 };
-  return unless defined &Win32::IsAdminUser;
-  return Win32::IsAdminUser();
-}
index c6481a6..2871cb5 100644 (file)
@@ -324,6 +324,7 @@ YAML::Syck
 YAML::Tiny
 dist/data-dumper/changes       Verbatim line length including indents exceeds 79 by    1
 dist/data-dumper/dumper.pm     ? Should you be using L<...> instead of 1
+dist/net-ping/lib/net/ping.pm  Apparent broken link    1
 ext/amiga-exec/exec.pm Verbatim line length including indents exceeds 79 by    1
 ext/dynaloader/dynaloader.pm   Verbatim line length including indents exceeds 79 by    1
 ext/hash-util/lib/hash/util.pm Verbatim line length including indents exceeds 79 by    2
@@ -362,7 +363,6 @@ pod/perlwin32.pod   Verbatim line length including indents exceeds 79 by    7
 porting/epigraphs.pod  Verbatim line length including indents exceeds 79 by    -1
 porting/release_managers_guide.pod     Verbatim line length including indents exceeds 79 by    1
 porting/todo.pod       ? Should you be using F<...> or maybe L<...> instead of 1
-utils/ptar     Verbatim paragraph in NAME section      1
 lib/benchmark.pm       Verbatim line length including indents exceeds 79 by    2
 lib/config.pod ? Should you be using L<...> instead of -1
 lib/perl5db.pl ? Should you be using L<...> instead of 1