swconfig: remove useless variables, return -1 on errors
authorJohn Crispin <john@openwrt.org>
Fri, 27 Dec 2013 21:15:20 +0000 (21:15 +0000)
committerJohn Crispin <john@openwrt.org>
Fri, 27 Dec 2013 21:15:20 +0000 (21:15 +0000)
spotted with cppcheck

Signed-off-by: Etienne CHAMPETIER <etienne.champetier@free.fr>
SVN-Revision: 39170

package/network/config/swconfig/src/cli.c
package/network/config/swconfig/src/swlib.c
package/network/config/swconfig/src/uci.c

index 2eb73bea38dbfc20226a74e8cd27f10f8188da2d..fa3dfdf6f2bda062e28c657f1c7c0ad97b86be3f 100644 (file)
@@ -178,7 +178,6 @@ swconfig_load_uci(struct switch_dev *dev, const char *name)
 {
        struct uci_context *ctx;
        struct uci_package *p = NULL;
-       struct uci_element *e;
        int ret = -1;
 
        ctx = uci_alloc_context();
@@ -206,7 +205,6 @@ int main(int argc, char **argv)
        struct switch_dev *dev;
        struct switch_attr *a;
        struct switch_val val;
-       int err;
        int i;
 
        int cmd = CMD_NONE;
@@ -289,6 +287,7 @@ int main(int argc, char **argv)
                if(!a)
                {
                        fprintf(stderr, "Unknown attribute \"%s\"\n", ckey);
+                       retval = -1;
                        goto out;
                }
        }
@@ -351,5 +350,5 @@ int main(int argc, char **argv)
 
 out:
        swlib_free_all(dev);
-       return 0;
+       return retval;
 }
index 7de3a604e0760010a1a9729dff26e11ed69efd08..122250256d7497af9dd4418e270ea43b9591a9cb 100644 (file)
@@ -206,7 +206,6 @@ store_val(struct nl_msg *msg, void *arg)
 {
        struct genlmsghdr *gnlh = nlmsg_data(nlmsg_hdr(msg));
        struct switch_val *val = arg;
-       struct switch_attr *attr = val->attr;
 
        if (!val)
                goto error;
@@ -669,11 +668,7 @@ done:
 static int
 list_switch(struct nl_msg *msg, void *arg)
 {
-       struct swlib_scan_arg *sa = arg;
        struct genlmsghdr *gnlh = nlmsg_data(nlmsg_hdr(msg));
-       struct switch_dev *dev;
-       const char *name;
-       const char *alias;
 
        if (nla_parse(tb, SWITCH_ATTR_MAX, genlmsg_attrdata(gnlh, 0), genlmsg_attrlen(gnlh, 0), NULL) < 0)
                goto done;
@@ -729,7 +724,6 @@ struct switch_dev *
 swlib_connect(const char *name)
 {
        struct swlib_scan_arg arg;
-       int err;
 
        if (!refcount) {
                if (swlib_priv_init() < 0)
index 893e3d1dd8f61e02678884f9beddb81dbd839fe8..bbeeb0311197767b39748f079f3eb705da9064fc 100644 (file)
@@ -66,7 +66,6 @@ swlib_map_settings(struct switch_dev *dev, int type, int port_vlan, struct uci_s
        struct switch_attr *attr;
        struct uci_element *e;
        struct uci_option *o;
-       int i;
 
        uci_foreach_element(&s->options, e) {
                o = uci_to_option(e);
@@ -109,7 +108,6 @@ skip:
 int swlib_apply_from_uci(struct switch_dev *dev, struct uci_package *p)
 {
        struct switch_attr *attr;
-       struct uci_context *ctx = p->ctx;
        struct uci_element *e;
        struct uci_section *s;
        struct uci_option *o;