libubus: Fix deletion from context's object AVL tree when removing object
authorBob Ham <bob.ham@tomltd.co.uk>
Thu, 9 Nov 2017 15:53:24 +0000 (15:53 +0000)
committerFelix Fietkau <nbd@nbd.name>
Mon, 13 Nov 2017 08:51:40 +0000 (09:51 +0100)
Objects are stored in the ubus context in an AVL tree.  An AVL tree
node contains a pointer to a key value.  For the ubus context, this
points to the id member of the object structure.  In
ubus_remove_object_cb, the id member is set to zero and then after,
avl_delete is called and fails.  To fix this, we call avl_delete
before setting the object id to zero.

Signed-off-by: Bob Ham <bob.ham@tomltd.co.uk>
libubus-obj.c

index a9972c5862eae0d62469e3010c346ed378877d12..9261b83246f26ea83167659970f71f504fa2df24 100644 (file)
@@ -237,12 +237,12 @@ static void ubus_remove_object_cb(struct ubus_request *req, int type, struct blo
        if (!attrbuf[UBUS_ATTR_OBJID])
                return;
 
+       avl_delete(&req->ctx->objects, &obj->avl);
+
        obj->id = 0;
 
        if (attrbuf[UBUS_ATTR_OBJTYPE] && obj->type)
                obj->type->id = 0;
-
-       avl_delete(&req->ctx->objects, &obj->avl);
 }
 
 int ubus_remove_object(struct ubus_context *ctx, struct ubus_object *obj)