the delete handle should return an error if no service is named instead of killing...
authorJohn Crispin <blogic@openwrt.org>
Tue, 9 Jul 2013 15:23:33 +0000 (17:23 +0200)
committerJohn Crispin <blogic@openwrt.org>
Tue, 9 Jul 2013 15:25:34 +0000 (17:25 +0200)
Signed-off-by: John Crispin <blogic@openwrt.org>
service.c

index ffbc58463ce95c6f6fbe4c67c4f40020fd2a209d..b5f4f3ed091745deefdb9eedd2657aaf4af8dea6 100644 (file)
--- a/service.c
+++ b/service.c
@@ -254,17 +254,14 @@ service_handle_delete(struct ubus_context *ctx, struct ubus_object *obj,
                    struct blob_attr *msg)
 {
        struct blob_attr *tb[__SERVICE_DEL_ATTR_MAX], *cur;
-       struct service *s, *tmp;
+       struct service *s;
        struct service_instance *in;
 
        blobmsg_parse(service_del_attrs, __SERVICE_DEL_ATTR_MAX, tb, blob_data(msg), blob_len(msg));
 
        cur = tb[SERVICE_DEL_ATTR_NAME];
-       if (!cur) {
-               avl_for_each_element_safe(&services, s, avl, tmp)
-                       service_delete(s);
-               return 0;
-       }
+       if (!cur)
+               return UBUS_STATUS_NOT_FOUND;
 
        s = avl_find_element(&services, blobmsg_data(cur), s, avl);
        if (!s)