ping: options -4 and -6 are mutually exclusive

Mutual exclusiveness should not stop use of -4 or -6 multiple times. Such usage is probably unintentional, but we can allow that without problems. Signed-off-by: Sami Kerola <[email protected]>
Mingli-Yu · Jul 22, 2019 · 50385e3 · 50385e3
1 parent 2306892
commit 50385e3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/ping.c b/ping.c
@@ -299,7 +299,7 @@ main(int argc, char **argv)
 		switch(ch) {
 		/* IPv4 specific options */
 		case '4':
-			if (hints.ai_family != AF_UNSPEC)
+			if (hints.ai_family == AF_INET6)
 				error(2, 0, _("only one -4 or -6 option may be specified"));
 			hints.ai_family = AF_INET;
 			break;
@@ -326,7 +326,7 @@ main(int argc, char **argv)
 			break;
 		/* IPv6 specific options */
 		case '6':
-			if (hints.ai_family != AF_UNSPEC)
+			if (hints.ai_family == AF_INET)
 				error(2, 0, _("only one -4 or -6 option may be specified"));
 			hints.ai_family = AF_INET6;
 			break;