projects
/
lttng-tools.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix: sessiond vs consumerd push/get metadata deadlock
[lttng-tools.git]
/
src
/
bin
/
lttng-sessiond
/
agent.c
diff --git
a/src/bin/lttng-sessiond/agent.c
b/src/bin/lttng-sessiond/agent.c
index 4a8473ebbe901ac147e15dbc2101edd6bd823362..32c261ca7529dba09e20544e8fd0debbbdfe07ab 100644
(file)
--- a/
src/bin/lttng-sessiond/agent.c
+++ b/
src/bin/lttng-sessiond/agent.c
@@
-125,7
+125,7
@@
static void destroy_event_agent_rcu(struct rcu_head *head)
struct agent_event *event =
caa_container_of(node, struct agent_event, node);
struct agent_event *event =
caa_container_of(node, struct agent_event, node);
-
free
(event);
+
agent_destroy_event
(event);
}
/*
}
/*
@@
-784,33
+784,37
@@
error:
}
/*
}
/*
- * Create a newly allocated agent event data structure.
If name is valid, it's
- *
copied into the created event
.
+ * Create a newly allocated agent event data structure.
+ *
Ownership of filter_expression is taken
.
*
* Return a new object else NULL on error.
*/
struct agent_event *agent_create_event(const char *name,
*
* Return a new object else NULL on error.
*/
struct agent_event *agent_create_event(const char *name,
- struct lttng_filter_bytecode *filter)
+ int loglevel, enum lttng_loglevel_type loglevel_type,
+ struct lttng_filter_bytecode *filter, char *filter_expression)
{
{
- struct agent_event *event;
+ struct agent_event *event
= NULL
;
DBG3("Agent create new event with name %s", name);
DBG3("Agent create new event with name %s", name);
- event = zmalloc(sizeof(*event));
- if (!event) {
+ if (!name) {
+ ERR("Failed to create agent event; no name provided.");
goto error;
}
goto error;
}
- if (name) {
- strncpy(event->name, name, sizeof(event->name));
- event->name[sizeof(event->name) - 1] = '\0';
- lttng_ht_node_init_str(&event->node, event->name);
+ event = zmalloc(sizeof(*event));
+ if (!event) {
+ goto error;
}
}
- if (filter) {
-
event->filter = filter
;
- }
+ strncpy(event->name, name, sizeof(event->name));
+
event->name[sizeof(event->name) - 1] = '\0'
;
+ lttng_ht_node_init_str(&event->node, event->name);
+ event->loglevel = loglevel;
+ event->loglevel_type = loglevel_type;
+ event->filter = filter;
+ event->filter_expression = filter_expression;
error:
return event;
}
error:
return event;
}
@@
-918,12
+922,12
@@
void agent_destroy_event(struct agent_event *event)
free(event->filter);
free(event->filter_expression);
free(event->filter);
free(event->filter_expression);
+ free(event->exclusion);
free(event);
}
/*
free(event);
}
/*
- * Destroy an agent completely. Note that the given pointer is NOT freed
- * thus a reference to static or stack data can be passed to this function.
+ * Destroy an agent completely.
*/
void agent_destroy(struct agent *agt)
{
*/
void agent_destroy(struct agent *agt)
{
@@
-962,6
+966,7
@@
void agent_destroy(struct agent *agt)
rcu_read_unlock();
ht_cleanup_push(agt->events);
rcu_read_unlock();
ht_cleanup_push(agt->events);
+ free(agt);
}
/*
}
/*
This page took
0.025952 seconds
and
4
git commands to generate.