projects
/
lttng-tools.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix: Memory leak of agent
[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 9ec98167eb5a3c4db7c6f428cca926ac6b2f42d6..b7d41f2f406ad948012666e45cde4872e93bc3ee 100644
(file)
--- a/
src/bin/lttng-sessiond/agent.c
+++ b/
src/bin/lttng-sessiond/agent.c
@@
-163,7
+163,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);
}
/*
}
/*
@@
-820,33
+820,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;
}
@@
-951,12
+955,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)
{
@@
-987,6
+991,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.024863 seconds
and
4
git commands to generate.