Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 6 Feb 2025 15:38:50 GMT
From:      Zhenlei Huang <zlei@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 99bcb54d8d91 - stable/14 - ip6addrctl(8): Strictly check the number of arguments
Message-ID:  <202502061538.516Fcom3002349@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/14 has been updated by zlei:

URL: https://cgit.FreeBSD.org/src/commit/?id=99bcb54d8d9138480001a262f080b7e8b0c24f58

commit 99bcb54d8d9138480001a262f080b7e8b0c24f58
Author:     Zhenlei Huang <zlei@FreeBSD.org>
AuthorDate: 2025-01-29 10:00:07 +0000
Commit:     Zhenlei Huang <zlei@FreeBSD.org>
CommitDate: 2025-02-06 15:38:03 +0000

    ip6addrctl(8): Strictly check the number of arguments
    
    The additional arguments are useless but may tempt user a wrong usage.
    
    Reviewed by:    dfr
    MFC after:      1 week
    Differential Revision:  https://reviews.freebsd.org/D48701
    
    (cherry picked from commit 13ce55b66fad2b2b4054a89a1a906f58dc33683b)
---
 usr.sbin/ip6addrctl/ip6addrctl.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/usr.sbin/ip6addrctl/ip6addrctl.c b/usr.sbin/ip6addrctl/ip6addrctl.c
index 146b2f1ad3e0..dd5df48048aa 100644
--- a/usr.sbin/ip6addrctl/ip6addrctl.c
+++ b/usr.sbin/ip6addrctl/ip6addrctl.c
@@ -75,22 +75,24 @@ static void flush_policy(void);
 int
 main(int argc, char *argv[])
 {
-	if (argc == 1 || strcasecmp(argv[1], "show") == 0) {
+	if (argc == 1 || (argc == 2 && strcasecmp(argv[1], "show") == 0)) {
 		get_policy();
 		dump_policy();
 	} else if (strcasecmp(argv[1], "add") == 0) {
-		if (argc < 5)
+		if (argc != 5)
 			usage();
 		add_policy(argv[2], argv[3], argv[4]);
 	} else if (strcasecmp(argv[1], "delete") == 0) {
-		if (argc < 3)
+		if (argc != 3)
 			usage();
 		delete_policy(argv[2]);
 	} else if (strcasecmp(argv[1], "flush") == 0) {
+		if (argc != 2)
+			usage();
 		get_policy();
 		flush_policy();
 	} else if (strcasecmp(argv[1], "install") == 0) {
-		if (argc < 3)
+		if (argc != 3)
 			usage();
 		configfile = argv[2];
 		make_policy_fromfile(configfile);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202502061538.516Fcom3002349>