]> git.ipfire.org Git - thirdparty/iproute2.git/commitdiff
tc: m_xt: Get rid of rargc in parse_ipt()
authorPhil Sutter <phil@nwl.cc>
Fri, 10 Jun 2016 11:42:05 +0000 (13:42 +0200)
committerStephen Hemminger <shemming@brocade.com>
Tue, 14 Jun 2016 21:35:56 +0000 (14:35 -0700)
No need to copy the passed parameter, it's changed only once right
before function return.

Signed-off-by: Phil Sutter <phil@nwl.cc>
tc/m_xt.c

index c42f3bda4e8f4ea72bb3f8e29730d6d64dbd4b8e..61fc437dfc907761bb02f1af95c3a32f9be9e2ad 100644 (file)
--- a/tc/m_xt.c
+++ b/tc/m_xt.c
@@ -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;