libubus: rework buffer allocation code
authorFelix Fietkau <nbd@openwrt.org>
Mon, 15 Sep 2014 13:36:03 +0000 (15:36 +0200)
committerFelix Fietkau <nbd@openwrt.org>
Mon, 15 Sep 2014 13:36:03 +0000 (15:36 +0200)
Signed-off-by: Felix Fietkau <nbd@openwrt.org>
libubus-io.c

index 55ddff5f62e47ddbb5ce428c2111a85a2fa52d01..fa53040575b7fef4a8401d6999d6db20f7650fd2 100644 (file)
@@ -231,6 +231,36 @@ static bool ubus_validate_hdr(struct ubus_msghdr *hdr)
        return true;
 }
 
+static bool alloc_msg_buf(struct ubus_context *ctx, int len)
+{
+       void *ptr;
+       int buf_len = ctx->msgbuf_data_len;
+       int rem;
+
+       if (!ctx->msgbuf.data)
+               buf_len = 0;
+
+       rem = (len % UBUS_MSG_CHUNK_SIZE);
+       if (rem > 0)
+               len += UBUS_MSG_CHUNK_SIZE - rem;
+
+       if (len < buf_len &&
+           ++ctx->msgbuf_reduction_counter > UBUS_MSGBUF_REDUCTION_INTERVAL) {
+               ctx->msgbuf_reduction_counter = 0;
+               buf_len = 0;
+       }
+
+       if (len <= buf_len)
+               return true;
+
+       ptr = realloc(ctx->msgbuf.data, len);
+       if (!ptr)
+               return false;
+
+       ctx->msgbuf.data = ptr;
+       return true;
+}
+
 static bool get_next_msg(struct ubus_context *ctx, int *recv_fd)
 {
        struct {
@@ -254,19 +284,7 @@ static bool get_next_msg(struct ubus_context *ctx, int *recv_fd)
                return false;
 
        len = blob_raw_len(&hdrbuf.data);
-       if (len > ctx->msgbuf_data_len)
-               ctx->msgbuf_reduction_counter = UBUS_MSGBUF_REDUCTION_INTERVAL;
-       else if (ctx->msgbuf_reduction_counter > 0 && len < UBUS_MSG_CHUNK_SIZE)
-               len = !--ctx->msgbuf_reduction_counter ? UBUS_MSG_CHUNK_SIZE : -1;
-       else
-               len = -1;
-
-       if (len > -1) {
-               ctx->msgbuf.data = realloc(ctx->msgbuf.data, len * sizeof(char));
-               if (ctx->msgbuf.data)
-                       ctx->msgbuf_data_len = len;
-       }
-       if (!ctx->msgbuf.data)
+       if (!alloc_msg_buf(ctx, len))
                return false;
 
        memcpy(&ctx->msgbuf.hdr, &hdrbuf.hdr, sizeof(hdrbuf.hdr));