X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=src%2Fbin%2Flttng-sessiond%2Fsession.c;h=b82a330e92da376aef158bddfd8448f925c7a603;hb=e8fcabef14e1e2ce8d159c68c5585931df1216b4;hp=a777b041428443fe856b4e7613920a377e7b494f;hpb=d022620a4a3ce49862b49527c5e29d7ba6cce743;p=lttng-tools.git diff --git a/src/bin/lttng-sessiond/session.c b/src/bin/lttng-sessiond/session.c index a777b0414..b82a330e9 100644 --- a/src/bin/lttng-sessiond/session.c +++ b/src/bin/lttng-sessiond/session.c @@ -16,6 +16,7 @@ */ #define _GNU_SOURCE +#define _LGPL_SOURCE #include #include #include @@ -23,6 +24,8 @@ #include #include #include +#include +#include #include #include @@ -49,6 +52,42 @@ static struct ltt_session_list ltt_session_list = { .next_uuid = 0, }; +/* These characters are forbidden in a session name. Used by validate_name. */ +static const char *forbidden_name_chars = "/"; + +/* + * Validate the session name for forbidden characters. + * + * Return 0 on success else -1 meaning a forbidden char. has been found. + */ +static int validate_name(const char *name) +{ + int ret; + char *tok, *tmp_name; + + assert(name); + + tmp_name = strdup(name); + if (!tmp_name) { + /* ENOMEM here. */ + ret = -1; + goto error; + } + + tok = strpbrk(tmp_name, forbidden_name_chars); + if (tok) { + DBG("Session name %s contains a forbidden character", name); + /* Forbidden character has been found. */ + ret = -1; + goto error; + } + ret = 0; + +error: + free(tmp_name); + return ret; +} + /* * Add a ltt_session structure to the global list. * @@ -124,7 +163,7 @@ void session_unlock(struct ltt_session *session) * NULL is returned. This must be called with the session lock held using * session_lock_list and session_unlock_list. */ -struct ltt_session *session_find_by_name(char *name) +struct ltt_session *session_find_by_name(const char *name) { struct ltt_session *iter; @@ -194,6 +233,22 @@ int session_create(char *name, uid_t uid, gid_t gid) goto error; } + ret = validate_name(name); + if (ret < 0) { + ret = LTTNG_ERR_SESSION_INVALID_CHAR; + goto error; + } + + ret = gethostname(new_session->hostname, sizeof(new_session->hostname)); + if (ret < 0) { + if (errno == ENAMETOOLONG) { + new_session->hostname[sizeof(new_session->hostname) - 1] = '\0'; + } else { + ret = LTTNG_ERR_FATAL; + goto error; + } + } + /* Init kernel session */ new_session->kernel_session = NULL; new_session->ust_session = NULL;