[PATCH uhttpd] ubus: rename JSON-RPC format related functions

Rafał Miłecki zajec5 at gmail.com
Wed Aug 5 02:58:45 EDT 2020


From: Rafał Miłecki <rafal at milecki.pl>

Use "_json_rpc_" in their names so it's clear they are related to the
JSON-RPC format. This cleans up code a bit and will allow adding more
formats in the future.

Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
---
 ubus.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/ubus.c b/ubus.c
index 062e973..c22e07a 100644
--- a/ubus.c
+++ b/ubus.c
@@ -189,7 +189,7 @@ static void uh_ubus_send_response(struct client *cl)
 		return ops->request_done(cl);
 }
 
-static void uh_ubus_init_response(struct client *cl)
+static void uh_ubus_init_json_rpc_response(struct client *cl)
 {
 	struct dispatch_ubus *du = &cl->dispatch.ubus;
 	struct json_object *obj = du->jsobj_cur, *obj2 = NULL;
@@ -205,11 +205,11 @@ static void uh_ubus_init_response(struct client *cl)
 		blobmsg_add_field(&buf, BLOBMSG_TYPE_UNSPEC, "id", NULL, 0);
 }
 
-static void uh_ubus_json_error(struct client *cl, enum rpc_error type)
+static void uh_ubus_json_rpc_error(struct client *cl, enum rpc_error type)
 {
 	void *c;
 
-	uh_ubus_init_response(cl);
+	uh_ubus_init_json_rpc_response(cl);
 	c = blobmsg_open_table(&buf, "error");
 	blobmsg_add_u32(&buf, "code", json_errors[type].code);
 	blobmsg_add_string(&buf, "message", json_errors[type].msg);
@@ -235,7 +235,7 @@ uh_ubus_request_cb(struct ubus_request *req, int ret)
 	int rem;
 
 	uloop_timeout_cancel(&du->timeout);
-	uh_ubus_init_response(cl);
+	uh_ubus_init_json_rpc_response(cl);
 	r = blobmsg_open_array(&buf, "result");
 	blobmsg_add_u32(&buf, "", ret);
 	blob_for_each_attr(cur, du->buf.head, rem)
@@ -251,7 +251,7 @@ uh_ubus_timeout_cb(struct uloop_timeout *timeout)
 	struct client *cl = container_of(du, struct client, dispatch.ubus);
 
 	ubus_abort_request(ctx, &du->req);
-	uh_ubus_json_error(cl, ERROR_TIMEOUT);
+	uh_ubus_json_rpc_error(cl, ERROR_TIMEOUT);
 }
 
 static void uh_ubus_close_fds(struct client *cl)
@@ -283,7 +283,7 @@ static void uh_ubus_request_free(struct client *cl)
 static void uh_ubus_single_error(struct client *cl, enum rpc_error type)
 {
 	uh_ubus_send_header(cl);
-	uh_ubus_json_error(cl, type);
+	uh_ubus_json_rpc_error(cl, type);
 	ops->request_done(cl);
 }
 
@@ -298,7 +298,7 @@ static void uh_ubus_send_request(struct client *cl, const char *sid, struct blob
 	blob_buf_init(&req, 0);
 	blobmsg_for_each_attr(cur, args, rem) {
 		if (!strcmp(blobmsg_name(cur), "ubus_rpc_session"))
-			return uh_ubus_json_error(cl, ERROR_PARAMS);
+			return uh_ubus_json_rpc_error(cl, ERROR_PARAMS);
 		blobmsg_add_blob(&req, cur);
 	}
 
@@ -308,7 +308,7 @@ static void uh_ubus_send_request(struct client *cl, const char *sid, struct blob
 	memset(&du->req, 0, sizeof(du->req));
 	ret = ubus_invoke_async(ctx, du->obj, du->func, req.head, &du->req);
 	if (ret)
-		return uh_ubus_json_error(cl, ERROR_INTERNAL);
+		return uh_ubus_json_rpc_error(cl, ERROR_INTERNAL);
 
 	du->req.data_cb = uh_ubus_request_data_cb;
 	du->req.complete_cb = uh_ubus_request_cb;
@@ -401,7 +401,7 @@ static void uh_ubus_send_list(struct client *cl, struct blob_attr *params)
 
 	uh_client_unref(cl);
 
-	uh_ubus_init_response(cl);
+	uh_ubus_init_json_rpc_response(cl);
 	blobmsg_add_blob(&buf, blob_data(data.buf->head));
 	uh_ubus_send_response(cl);
 }
@@ -564,7 +564,7 @@ static void uh_ubus_handle_request_object(struct client *cl, struct json_object
 	}
 
 error:
-	uh_ubus_json_error(cl, err);
+	uh_ubus_json_rpc_error(cl, err);
 out:
 	if (data.params)
 		free(data.params);
-- 
2.27.0




More information about the openwrt-devel mailing list