Use inet_pton() rather than inet_addr().

Modern UN*Xes should have it, and MSVC whines about inet_addr() being
deprecated in the Appveyor builds.

diff --git a/tests/filtertest.c b/tests/filtertest.c
index 91f942852231ec5eee43b09822dc2dbd43f2073c..746cc9440778994a058a43fc99ffd800442f1d95 100644 (file)
--- a/tests/filtertest.c
+++ b/tests/filtertest.c
@@ -240,10 +240,21 @@ main(int argc, char **argv)
                case 'm': {
                        in_addr_t addr;
 
-                       addr = inet_addr(optarg);
-                       if (addr == (in_addr_t)(-1))
+                       switch (inet_pton(AF_INET, optarg, &addr)) {
+
+                       case 0:                        
                                error("invalid netmask %s", optarg);
-                       netmask = addr;
+                               break;
+
+                       case -1:
+                               error("invalid netmask %s: %s", optarg,
+                                   pcap_strerror(errno));
+                               break;
+
+                       case 1:
+                               netmask = addr;
+                               break;
+                       }
                        break;
                }