X-Git-Url: http://git.efficios.com/?a=blobdiff_plain;f=src%2Fcommon%2Frelayd%2Frelayd.c;h=fade4b11ed56bc77750bfaed592008c9655e8b33;hb=246777db6078ff9dd8046d5701fc12cb34547402;hp=47db4081ae41bf55e7ca5d7c5fa6ab8194e37f9f;hpb=6c1c0768320135c6936c371b09731851b508c023;p=lttng-tools.git diff --git a/src/common/relayd/relayd.c b/src/common/relayd/relayd.c index 47db4081a..fade4b11e 100644 --- a/src/common/relayd/relayd.c +++ b/src/common/relayd/relayd.c @@ -15,7 +15,6 @@ * Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. */ -#define _GNU_SOURCE #define _LGPL_SOURCE #include #include @@ -130,7 +129,15 @@ static int relayd_create_session_2_4(struct lttcomm_relayd_sock *rsock, int ret; struct lttcomm_relayd_create_session_2_4 msg; + if (strlen(session_name) >= sizeof(msg.session_name)) { + ret = -1; + goto error; + } strncpy(msg.session_name, session_name, sizeof(msg.session_name)); + if (strlen(hostname) >= sizeof(msg.hostname)) { + ret = -1; + goto error; + } strncpy(msg.hostname, hostname, sizeof(msg.hostname)); msg.live_timer = htobe32(session_live_timer); msg.snapshot = htobe32(snapshot); @@ -248,7 +255,15 @@ int relayd_add_stream(struct lttcomm_relayd_sock *rsock, const char *channel_nam /* Compat with relayd 2.1 */ if (rsock->minor == 1) { memset(&msg, 0, sizeof(msg)); + if (strlen(channel_name) >= sizeof(msg.channel_name)) { + ret = -1; + goto error; + } strncpy(msg.channel_name, channel_name, sizeof(msg.channel_name)); + if (strlen(pathname) >= sizeof(msg.pathname)) { + ret = -1; + goto error; + } strncpy(msg.pathname, pathname, sizeof(msg.pathname)); /* Send command */ @@ -259,7 +274,15 @@ int relayd_add_stream(struct lttcomm_relayd_sock *rsock, const char *channel_nam } else { memset(&msg_2_2, 0, sizeof(msg_2_2)); /* Compat with relayd 2.2+ */ + if (strlen(channel_name) >= sizeof(msg_2_2.channel_name)) { + ret = -1; + goto error; + } strncpy(msg_2_2.channel_name, channel_name, sizeof(msg_2_2.channel_name)); + if (strlen(pathname) >= sizeof(msg_2_2.pathname)) { + ret = -1; + goto error; + } strncpy(msg_2_2.pathname, pathname, sizeof(msg_2_2.pathname)); msg_2_2.tracefile_size = htobe64(tracefile_size); msg_2_2.tracefile_count = htobe64(tracefile_count);