summaryrefslogtreecommitdiffstats
path: root/lib/filter.c
diff options
context:
space:
mode:
authorDaniel Walton <dwalton@cumulusnetworks.com>2016-09-30 02:16:31 +0200
committerDaniel Walton <dwalton@cumulusnetworks.com>2016-09-30 02:16:31 +0200
commit58749582a9a2c5930d907d88a3ad0d0c5c24f627 (patch)
treeca4b5aed981645901812b40478bb0573e0034230 /lib/filter.c
parentisisd: scrub argc CHECK ME's, refactor general (diff)
downloadfrr-58749582a9a2c5930d907d88a3ad0d0c5c24f627.tar.xz
frr-58749582a9a2c5930d907d88a3ad0d0c5c24f627.zip
all: scrubbed some argc CHECK MEs
Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>
Diffstat (limited to 'lib/filter.c')
-rw-r--r--lib/filter.c10
1 files changed, 4 insertions, 6 deletions
diff --git a/lib/filter.c b/lib/filter.c
index fd80b2a63..501b28f2d 100644
--- a/lib/filter.c
+++ b/lib/filter.c
@@ -1480,8 +1480,8 @@ DEFUN (access_list_remark,
"Access list entry comment\n"
"Comment up to 100 characters\n")
{
- /* CHECK ME argc referenced below */
int idx_acl = 1;
+ int idx_remark = 3;
struct access_list *access;
access = access_list_get (AFI_IP, argv[idx_acl]->arg);
@@ -1491,7 +1491,7 @@ DEFUN (access_list_remark,
XFREE (MTYPE_TMP, access->remark);
access->remark = NULL;
}
- access->remark = argv_concat(argv, argc, 1);
+ access->remark = argv_concat(argv, argc, idx_remark);
return CMD_SUCCESS;
}
@@ -1526,7 +1526,6 @@ DEFUN (no_access_list_remark_comment,
"Access list entry comment\n"
"Comment up to 100 characters\n")
{
- /* CHECK ME argc referenced below */
return no_access_list_remark (self, vty, argc, argv);
}
@@ -1675,8 +1674,8 @@ DEFUN (ipv6_access_list_remark,
"Access list entry comment\n"
"Comment up to 100 characters\n")
{
- /* CHECK ME argc referenced below */
int idx_word = 2;
+ int idx_line = 4;
struct access_list *access;
access = access_list_get (AFI_IP6, argv[idx_word]->arg);
@@ -1686,7 +1685,7 @@ DEFUN (ipv6_access_list_remark,
XFREE (MTYPE_TMP, access->remark);
access->remark = NULL;
}
- access->remark = argv_concat(argv, argc, 1);
+ access->remark = argv_concat(argv, argc, idx_line);
return CMD_SUCCESS;
}
@@ -1715,7 +1714,6 @@ DEFUN (no_ipv6_access_list_remark_comment,
"Access list entry comment\n"
"Comment up to 100 characters\n")
{
- /* CHECK ME argc referenced below */
return no_ipv6_access_list_remark (self, vty, argc, argv);
}