From 4fc532c8a55ba8217ad67d7fd47c5eb9a8aba044 Mon Sep 17 00:00:00 2001 From: Alexander Van Parys Date: Wed, 30 Jun 2021 20:12:46 +0200 Subject: ubusd: fix tx_queue linked list usage Use the correct parameters for list_add_tail when enqueueing a message in the tx_queue. Previously, list_add_tail(list, entry) was used instead of list_add_tail(entry, list). Due to this, the list would only contain the latest entry, and previously inserted entries were left dangling. Reset the tx_queue offset after a message from the tx_queue has been sent completely. Signed-off-by: Alexander Van Parys --- ubusd.c | 2 +- ubusd_main.c | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/ubusd.c b/ubusd.c index 0e1b0c9..1d76b72 100644 --- a/ubusd.c +++ b/ubusd.c @@ -154,7 +154,7 @@ static void ubus_msg_enqueue(struct ubus_client *cl, struct ubus_msg_buf *ub) INIT_LIST_HEAD(&ubl->list); ubl->msg = ubus_msg_ref(ub); - list_add_tail(&cl->tx_queue, &ubl->list); + list_add_tail(&ubl->list, &cl->tx_queue); cl->txq_len += ub->len; } diff --git a/ubusd_main.c b/ubusd_main.c index 4ea5245..d454b1a 100644 --- a/ubusd_main.c +++ b/ubusd_main.c @@ -78,6 +78,7 @@ static void client_cb(struct uloop_fd *sock, unsigned int events) if (cl->txq_ofs < ub->len + sizeof(ub->hdr)) break; + cl->txq_ofs = 0; ubus_msg_list_free(ubl); } -- cgit v1.2.1