aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJo-Philipp Wich <jo@mein.io>2016-06-11 00:51:22 +0200
committerJo-Philipp Wich <jo@mein.io>2016-06-11 00:51:22 +0200
commitdd182011e1acabc94169b85f3bc63efbab72ddd4 (patch)
tree6654fba853fa3c2e8b9b04e49fdc71877e296c27
parente8150364608e38e6b30ea0859ffa17ccb1fcc890 (diff)
downloadupstream-dd182011e1acabc94169b85f3bc63efbab72ddd4.tar.gz
upstream-dd182011e1acabc94169b85f3bc63efbab72ddd4.tar.bz2
upstream-dd182011e1acabc94169b85f3bc63efbab72ddd4.zip
swconfig: improve failure reporting
Report the translated error to the user if a get/set netlink operation failed. Signed-off-by: Jo-Philipp Wich <jo@mein.io>
-rw-r--r--package/network/config/swconfig/Makefile2
-rw-r--r--package/network/config/swconfig/src/cli.c12
2 files changed, 7 insertions, 7 deletions
diff --git a/package/network/config/swconfig/Makefile b/package/network/config/swconfig/Makefile
index 2856742fcd..f4ba3a9d69 100644
--- a/package/network/config/swconfig/Makefile
+++ b/package/network/config/swconfig/Makefile
@@ -8,7 +8,7 @@
include $(TOPDIR)/rules.mk
PKG_NAME:=swconfig
-PKG_RELEASE:=10
+PKG_RELEASE:=11
PKG_MAINTAINER:=Felix Fietkau <nbd@nbd.name>
PKG_LICENSE:=GPL-2.0
diff --git a/package/network/config/swconfig/src/cli.c b/package/network/config/swconfig/src/cli.c
index 331f3d5dac..eab6c64742 100644
--- a/package/network/config/swconfig/src/cli.c
+++ b/package/network/config/swconfig/src/cli.c
@@ -335,10 +335,10 @@ int main(int argc, char **argv)
if(cvlan > -1)
cport = cvlan;
- if(swlib_set_attr_string(dev, a, cport, cvalue) < 0)
+ retval = swlib_set_attr_string(dev, a, cport, cvalue);
+ if (retval < 0)
{
- fprintf(stderr, "failed\n");
- retval = -1;
+ nl_perror(-retval, "Failed to set attribute");
goto out;
}
break;
@@ -347,10 +347,10 @@ int main(int argc, char **argv)
val.port_vlan = cvlan;
if(cport > -1)
val.port_vlan = cport;
- if(swlib_get_attr(dev, a, &val) < 0)
+ retval = swlib_get_attr(dev, a, &val);
+ if (retval < 0)
{
- fprintf(stderr, "failed\n");
- retval = -1;
+ nl_perror(-retval, "Failed to get attribute");
goto out;
}
print_attr_val(a, &val);