From 95a29ab808f93c74a910b2fc8ee25833bfbc46d7 Mon Sep 17 00:00:00 2001 From: David Goulet Date: Wed, 12 Mar 2014 16:31:32 -0400 Subject: [PATCH] Fix: missing memset 0 and NULL byte in string Signed-off-by: David Goulet --- src/bin/lttng-sessiond/save.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/bin/lttng-sessiond/save.c b/src/bin/lttng-sessiond/save.c index 22fab9c05..fd6c5d894 100644 --- a/src/bin/lttng-sessiond/save.c +++ b/src/bin/lttng-sessiond/save.c @@ -1410,6 +1410,7 @@ int save_session(struct ltt_session *session, assert(creds); session_name_len = strlen(session->name); + memset(config_file_path, 0, sizeof(config_file_path)); if (!session_access_ok(session, LTTNG_SOCK_GET_UID_CRED(creds), @@ -1420,6 +1421,7 @@ int save_session(struct ltt_session *session, provided_path = lttng_save_session_attr_get_output_url(attr); if (provided_path) { + DBG3("Save session in provided path %s", provided_path); len = strlen(provided_path); if (len >= sizeof(config_file_path)) { ret = LTTNG_ERR_SET_URL; @@ -1472,6 +1474,8 @@ int save_session(struct ltt_session *session, strncpy(config_file_path + len, session->name, session_name_len); len += session_name_len; strcpy(config_file_path + len, DEFAULT_SESSION_CONFIG_FILE_EXTENSION); + len += sizeof(DEFAULT_SESSION_CONFIG_FILE_EXTENSION); + config_file_path[len] = '\0'; if (!access(config_file_path, F_OK) && !attr->overwrite) { /* A file with the same name already exists, skip */ -- 2.34.1