X-Git-Url: http://git.archive.openwrt.org/?p=project%2Fubus.git;a=blobdiff_plain;f=ubusd_event.c;h=85031a6fc150c94847b1db9b3381f8ff485ea72b;hp=aae2e0d1452d418a5bd14698ea8a0652b1a15fc6;hb=b79e808944675040ec2cbc2534592a5d4e62477b;hpb=0611a9248c525c472fd3ff0f067cfb051c0b4962

diff --git a/ubusd_event.c b/ubusd_event.c
index aae2e0d..85031a6 100644
--- a/ubusd_event.c
+++ b/ubusd_event.c
@@ -1,43 +1,35 @@
+/*
+ * Copyright (C) 2011 Felix Fietkau <nbd@openwrt.org>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License version 2.1
+ * as published by the Free Software Foundation
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
 #include <arpa/inet.h>
 #include "ubusd.h"
 
 static struct avl_tree patterns;
-static LIST_HEAD(catch_all);
 static struct ubus_object *event_obj;
 static int event_seq = 0;
-static int obj_event_seq = 0;
-
-enum evs_type {
-	EVS_PATTERN,
-	EVS_CATCHALL
-};
+static int obj_event_seq = 1;
 
 struct event_source {
 	struct list_head list;
 	struct ubus_object *obj;
-	enum evs_type type;
-	union {
-		struct {
-			struct avl_node avl;
-			bool partial;
-		} pattern;
-		struct {
-			struct list_head list;
-		} catchall;
-	};
+	struct avl_node avl;
+	bool partial;
 };
 
 static void ubusd_delete_event_source(struct event_source *evs)
 {
 	list_del(&evs->list);
-	switch (evs->type) {
-	case EVS_PATTERN:
-		avl_delete(&patterns, &evs->pattern.avl);
-		break;
-	case EVS_CATCHALL:
-		list_del(&evs->catchall.list);
-		break;
-	}
+	avl_delete(&patterns, &evs->avl);
 	free(evs);
 }
 
@@ -62,40 +54,18 @@ static struct blobmsg_policy evr_policy[] = {
 	[EVREG_OBJECT] = { .name = "object", .type = BLOBMSG_TYPE_INT32 },
 };
 
-
-static struct event_source *ubusd_alloc_event_source(struct ubus_object *obj, enum evs_type type, int datalen)
-{
-	struct event_source *evs;
-
-	evs = calloc(1, sizeof(*evs) + datalen);
-	list_add(&evs->list, &obj->events);
-	evs->obj = obj;
-	evs->type = type;
-	return evs;
-}
-
-static int ubusd_alloc_catchall(struct ubus_object *obj)
-{
-	struct event_source *evs;
-
-	evs = ubusd_alloc_event_source(obj, EVS_CATCHALL, 0);
-	list_add(&evs->catchall.list, &catch_all);
-
-	return 0;
-}
-
 static int ubusd_alloc_event_pattern(struct ubus_client *cl, struct blob_attr *msg)
 {
 	struct event_source *ev;
 	struct ubus_object *obj;
 	struct blob_attr *attr[EVREG_LAST];
-	char *pattern;
+	char *pattern, *name;
 	uint32_t id;
 	bool partial = false;
 	int len;
 
 	blobmsg_parse(evr_policy, EVREG_LAST, attr, blob_data(msg), blob_len(msg));
-	if (!attr[EVREG_OBJECT])
+	if (!attr[EVREG_OBJECT] || !attr[EVREG_PATTERN])
 		return UBUS_STATUS_INVALID_ARGUMENT;
 
 	id = blobmsg_get_u32(attr[EVREG_OBJECT]);
@@ -109,9 +79,6 @@ static int ubusd_alloc_event_pattern(struct ubus_client *cl, struct blob_attr *m
 	if (obj->client != cl)
 		return UBUS_STATUS_PERMISSION_DENIED;
 
-	if (!attr[EVREG_PATTERN])
-		return ubusd_alloc_catchall(obj);
-
 	pattern = blobmsg_data(attr[EVREG_PATTERN]);
 
 	len = strlen(pattern);
@@ -121,11 +88,17 @@ static int ubusd_alloc_event_pattern(struct ubus_client *cl, struct blob_attr *m
 		len--;
 	}
 
-	ev = ubusd_alloc_event_source(obj, EVS_PATTERN, len + 1);
-	ev->pattern.partial = partial;
-	ev->pattern.avl.key = (void *) (ev + 1);
-	strcpy(ev->pattern.avl.key, pattern);
-	avl_insert(&patterns, &ev->pattern.avl);
+	ev = calloc(1, sizeof(*ev) + len + 1);
+	if (!ev)
+		return UBUS_STATUS_NO_DATA;
+
+	list_add(&ev->list, &obj->events);
+	ev->obj = obj;
+	ev->partial = partial;
+	name = (char *) (ev + 1);
+	strcpy(name, pattern);
+	ev->avl.key = name;
+	avl_insert(&patterns, &ev->avl);
 
 	return 0;
 }
@@ -161,7 +134,7 @@ static void ubusd_send_event_msg(struct ubus_msg_buf **ub, struct ubus_client *c
 	ubus_msg_send(obj->client, *ub, false);
 }
 
-bool strmatch_len(const char *s1, const char *s2, int *len)
+static bool strmatch_len(const char *s1, const char *s2, int *len)
 {
 	for (*len = 0; s1[*len] == s2[*len]; (*len)++)
 		if (!s1[*len])
@@ -177,9 +150,6 @@ static int ubusd_send_event(struct ubus_client *cl, const char *id,
 	struct event_source *ev;
 	int match_len = 0;
 
-	list_for_each_entry(ev, &catch_all, catchall.list)
-		ubusd_send_event_msg(&ub, cl, ev->obj, id, fill_cb, cb_priv);
-
 	obj_event_seq++;
 
 	/*
@@ -187,8 +157,8 @@ static int ubusd_send_event(struct ubus_client *cl, const char *id,
 	 * matching entries as long as the number of matching characters
 	 * between the pattern string and our string is monotonically increasing.
 	 */
-	avl_for_each_element(&patterns, ev, pattern.avl) {
-		const char *key = ev->pattern.avl.key;
+	avl_for_each_element(&patterns, ev, avl) {
+		const char *key = ev->avl.key;
 		int cur_match_len;
 		bool full_match;
 
@@ -199,7 +169,7 @@ static int ubusd_send_event(struct ubus_client *cl, const char *id,
 		match_len = cur_match_len;
 
 		if (!full_match) {
-			if (!ev->pattern.partial)
+			if (!ev->partial)
 				continue;
 
 			if (match_len != strlen(key))