projects
/
lttng-tools.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix: Cleanup UST app session on ustctl create session error
[lttng-tools.git]
/
src
/
bin
/
lttng-sessiond
/
ust-app.c
diff --git
a/src/bin/lttng-sessiond/ust-app.c
b/src/bin/lttng-sessiond/ust-app.c
index c93f93dc9add8333c07470688ae95f14a8f1ec80..96f8eb0b3ead316c0defa96905ff40cfcccb3136 100644
(file)
--- a/
src/bin/lttng-sessiond/ust-app.c
+++ b/
src/bin/lttng-sessiond/ust-app.c
@@
-976,9
+976,10
@@
static struct ust_app_session *create_ust_app_session(
ret = ustctl_create_session(app->sock);
if (ret < 0) {
ERR("Creating session for app pid %d", app->pid);
ret = ustctl_create_session(app->sock);
if (ret < 0) {
ERR("Creating session for app pid %d", app->pid);
+ delete_ust_app_session(-1, ua_sess);
/* This means that the tracer is gone... */
ua_sess = (void*) -1UL;
/* This means that the tracer is gone... */
ua_sess = (void*) -1UL;
- goto e
rror
;
+ goto e
nd
;
}
ua_sess->handle = ret;
}
ua_sess->handle = ret;
@@
-993,11
+994,6
@@
static struct ust_app_session *create_ust_app_session(
end:
health_code_update(&health_thread_cmd);
return ua_sess;
end:
health_code_update(&health_thread_cmd);
return ua_sess;
-
-error:
- delete_ust_app_session(-1, ua_sess);
- health_code_update(&health_thread_cmd);
- return NULL;
}
/*
}
/*
@@
-1702,21
+1698,22
@@
error:
void ust_app_clean_list(void)
{
int ret;
void ust_app_clean_list(void)
{
int ret;
+ struct ust_app *app;
struct lttng_ht_iter iter;
struct lttng_ht_iter iter;
- struct lttng_ht_node_ulong *node;
DBG2("UST app cleaning registered apps hash table");
rcu_read_lock();
DBG2("UST app cleaning registered apps hash table");
rcu_read_lock();
- cds_lfht_for_each_entry(ust_app_ht->ht, &iter.iter,
node,
node) {
+ cds_lfht_for_each_entry(ust_app_ht->ht, &iter.iter,
app, pid_n.
node) {
ret = lttng_ht_del(ust_app_ht, &iter);
assert(!ret);
ret = lttng_ht_del(ust_app_ht, &iter);
assert(!ret);
- call_rcu(&
node->
head, delete_ust_app_rcu);
+ call_rcu(&
app->pid_n.
head, delete_ust_app_rcu);
}
/* Cleanup socket hash table */
}
/* Cleanup socket hash table */
- cds_lfht_for_each_entry(ust_app_ht_by_sock->ht, &iter.iter, node, node) {
+ cds_lfht_for_each_entry(ust_app_ht_by_sock->ht, &iter.iter, app,
+ sock_n.node) {
ret = lttng_ht_del(ust_app_ht_by_sock, &iter);
assert(!ret);
}
ret = lttng_ht_del(ust_app_ht_by_sock, &iter);
assert(!ret);
}
This page took
0.02372 seconds
and
4
git commands to generate.