From 3688e3a3c27e71e4e20c47dc749a3fcb1fc9be20 Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Mon, 25 Jun 2012 21:07:35 +0200 Subject: [PATCH] remove outer memdup --- service.c | 21 ++++++++------------- 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/service.c b/service.c index 56e0feb..da7ec9b 100644 --- a/service.c +++ b/service.c @@ -53,10 +53,17 @@ static struct service * service_alloc(const char *name) { struct service *s; + char *new_name; + + s = calloc(1, sizeof(*s) + strlen(name) + 1); + + new_name = (char *) s + 1; + strcpy(new_name, name); - s = calloc(1, sizeof(*s)); vlist_init(&s->instances, avl_strcmp, service_instance_update); s->instances.keep_old = true; + s->name = new_name; + s->avl.key = s->name; return s; } @@ -78,15 +85,9 @@ static const struct blobmsg_policy service_set_attrs[__SERVICE_SET_MAX] = { static int service_update(struct service *s, struct blob_attr *config, struct blob_attr **tb) { - struct blob_attr *old_config = s->config; struct blob_attr *cur; int rem; - /* only the pointer changes, the content stays the same, - * no avl update necessary */ - s->name = s->avl.key = blobmsg_data(tb[SERVICE_SET_NAME]); - s->config = config; - if (tb[SERVICE_SET_INSTANCES]) { vlist_update(&s->instances); blobmsg_for_each_attr(cur, tb[SERVICE_SET_INSTANCES], rem) { @@ -95,8 +96,6 @@ service_update(struct service *s, struct blob_attr *config, struct blob_attr **t vlist_flush(&s->instances); } - free(old_config); - return 0; } @@ -129,10 +128,6 @@ service_handle_set(struct ubus_context *ctx, struct ubus_object *obj, const char *name; int ret = UBUS_STATUS_INVALID_ARGUMENT; - msg = blob_memdup(msg); - if (!msg) - return UBUS_STATUS_UNKNOWN_ERROR; - blobmsg_parse(service_set_attrs, __SERVICE_SET_MAX, tb, blob_data(msg), blob_len(msg)); cur = tb[SERVICE_ATTR_NAME]; if (!cur) -- 2.25.1