cli: fix return value of package_cmd().
authorYousong Zhou <yszhou4tech@gmail.com>
Wed, 18 Mar 2015 04:44:44 +0000 (12:44 +0800)
committerFelix Fietkau <nbd@openwrt.org>
Thu, 26 Mar 2015 20:38:05 +0000 (21:38 +0100)
It's a bug revealed by commit 446e774 "cli: properly unload package
before quit".  The current code would exit with value 1 even if
uci_export() succceeded.

Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
cli.c

diff --git a/cli.c b/cli.c
index b647f777baa595597229428ba07164ad86a42f31..557472ecf42100e9bf6d9eee52a1ef9b66f308c3 100644 (file)
--- a/cli.c
+++ b/cli.c
@@ -305,10 +305,13 @@ static int package_cmd(int cmd, char *tuple)
                }
                if (uci_commit(ctx, &ptr.p, false) != UCI_OK) {
                        cli_perror();
+                       goto out;
                }
                break;
        case CMD_EXPORT:
-               uci_export(ctx, stdout, ptr.p, true);
+               if (uci_export(ctx, stdout, ptr.p, true) != UCI_OK) {
+                       goto out;
+               }
                break;
        case CMD_SHOW:
                if (!(ptr.flags & UCI_LOOKUP_COMPLETE)) {
@@ -333,6 +336,8 @@ static int package_cmd(int cmd, char *tuple)
                break;
        }
 
+       ret = 0;
+
 out:
        if (ptr.p)
                uci_unload(ctx, ptr.p);