cli: use UBUS_STATUS_PARSE_ERROR
authorStijn Tintel <stijn@linux-ipv6.be>
Fri, 18 Feb 2022 10:51:03 +0000 (12:51 +0200)
committerStijn Tintel <stijn@linux-ipv6.be>
Mon, 28 Feb 2022 14:07:44 +0000 (16:07 +0200)
Use the newly introduced UBUS_STATUS_PARSE_ERROR status message in
ubus_cli_call() and ubus_cli_send().

Signed-off-by: Stijn Tintel <stijn@linux-ipv6.be>
Acked-by: Petr Štetiar <ynezz@true.cz>
Acked-by: Jo-Philipp Wich <jo@mein.io>
cli.c

diff --git a/cli.c b/cli.c
index 81591ec44d11aa053d3461d774d58e59b0bfbbe0..3b67047fa9e81e3641ca175a8ab35640338eed13 100644 (file)
--- a/cli.c
+++ b/cli.c
@@ -142,9 +142,7 @@ static int ubus_cli_call(struct ubus_context *ctx, int argc, char **argv)
 
        blob_buf_init(&b, 0);
        if (argc == 3 && !blobmsg_add_json_from_string(&b, argv[2])) {
-               if (!simple_output)
-                       fprintf(stderr, "Failed to parse message data\n");
-               return -1;
+               return UBUS_STATUS_PARSE_ERROR;
        }
 
        ret = ubus_lookup_id(ctx, argv[0], &id);
@@ -268,9 +266,7 @@ static int ubus_cli_send(struct ubus_context *ctx, int argc, char **argv)
        blob_buf_init(&b, 0);
 
        if (argc == 2 && !blobmsg_add_json_from_string(&b, argv[1])) {
-               if (!simple_output)
-                       fprintf(stderr, "Failed to parse message data\n");
-               return -1;
+               return UBUS_STATUS_PARSE_ERROR;
        }
 
        return ubus_send_event(ctx, argv[0], b.head);