matthias
/
iproute2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

tc: m_xt: Get rid of rargc in parse_ipt() 

No need to copy the passed parameter, it's changed only once right
before function return.

Signed-off-by: Phil Sutter <phil@nwl.cc>
This commit is contained in:
Phil Sutter 2016-06-10 13:42:05 +02:00 committed by Stephen Hemminger
parent b0ba018576
commit ab8f52fc4a
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
tc/m_xt.c
View File

@ -123,7 +123,6 @@ static int parse_ipt(struct action_util *a, int *argc_p,
struct rtattr *tail;
int c;
int rargc = *argc_p;
char **argv = *argv_p;
int argc = 0, iargc = 0;
char k[16];
@ -140,7 +139,7 @@ static int parse_ipt(struct action_util *a, int *argc_p,
{
int i;
for (i = 0; i < rargc; i++) {
for (i = 0; i < *argc_p; i++) {
if (NULL == argv[i] || 0 == strcmp(argv[i], "action")) {
break;
}
@ -149,7 +148,7 @@ static int parse_ipt(struct action_util *a, int *argc_p,
}
if (argc <= 2) {
fprintf(stderr, "bad arguments to ipt %d vs %d\n", argc, rargc);
fprintf(stderr, "bad arguments to ipt %d vs %d\n", argc, *argc_p);
return -1;
}
@ -274,7 +273,7 @@ static int parse_ipt(struct action_util *a, int *argc_p,
argc -= optind;
argv += optind;
*argc_p = rargc - iargc;
*argc_p -= iargc;
*argv_p = argv;
optind = 0;