X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=src%2Fcommon%2Frelayd%2Frelayd.c;h=22a1c44f800a4bb5302fe12d40c479a35e32e4c6;hb=0a23a07d7db04d7e9f1fcf08a5824fccbf2b53da;hp=7b11c81f63ed8ebfead0d36a919f4e0ec2e06dce;hpb=ab5be9fa2eb5ba9600a82cd18fd3cfcbac69169a;p=lttng-tools.git

diff --git a/src/common/relayd/relayd.c b/src/common/relayd/relayd.c
index 7b11c81f6..22a1c44f8 100644
--- a/src/common/relayd/relayd.c
+++ b/src/common/relayd/relayd.c
@@ -499,7 +499,6 @@ int relayd_add_stream(struct lttcomm_relayd_sock *rsock, const char *channel_nam
 	int ret;
 	struct lttcomm_relayd_status_stream reply;
 	char pathname[RELAYD_COMM_LTTNG_PATH_MAX];
-	char *separator;
 
 	/* Code flow error. Safety net. */
 	assert(rsock);
@@ -510,32 +509,36 @@ int relayd_add_stream(struct lttcomm_relayd_sock *rsock, const char *channel_nam
 
 	DBG("Relayd adding stream for channel name %s", channel_name);
 
-	if (_pathname[0] == '\0') {
-		separator = "";
-	} else {
-		separator = "/";
-	}
-	ret = snprintf(pathname, RELAYD_COMM_LTTNG_PATH_MAX, "%s%s%s",
-			domain_name, separator, _pathname);
-	if (ret <= 0 || ret >= RELAYD_COMM_LTTNG_PATH_MAX) {
-		ERR("stream path too long.");
-		ret = -1;
-		goto error;
-	}
-
 	/* Compat with relayd 2.1 */
 	if (rsock->minor == 1) {
 		/* For 2.1 */
-		ret = relayd_add_stream_2_1(rsock, channel_name, pathname);
+		ret = relayd_add_stream_2_1(rsock, channel_name, _pathname);
 	
 	} else if (rsock->minor > 1 && rsock->minor < 11) {
 		/* From 2.2 to 2.10 */
-		ret = relayd_add_stream_2_2(rsock, channel_name, pathname,
+		ret = relayd_add_stream_2_2(rsock, channel_name, _pathname,
 				tracefile_size, tracefile_count);
 	} else {
+		const char *separator;
 		enum lttng_trace_chunk_status chunk_status;
 		uint64_t chunk_id;
 
+		if (_pathname[0] == '\0') {
+			separator = "";
+		} else {
+			separator = "/";
+		}
+
+		ret = snprintf(pathname, RELAYD_COMM_LTTNG_PATH_MAX, "%s%s%s",
+				domain_name, separator, _pathname);
+		if (ret <= 0 || ret >= RELAYD_COMM_LTTNG_PATH_MAX) {
+			ERR("Failed to format stream path: %s",
+					ret <= 0 ? "formatting error" :
+							"path exceeds maximal allowed length");
+			ret = -1;
+			goto error;
+		}
+
 		chunk_status = lttng_trace_chunk_get_id(trace_chunk,
 				&chunk_id);
 		assert(chunk_status == LTTNG_TRACE_CHUNK_STATUS_OK);
@@ -1258,7 +1261,7 @@ int relayd_rotate_streams(struct lttcomm_relayd_sock *sock,
 					position->rotate_at_seq_num),
 		};
 
-		DBG("Rotate stream %" PRIu64 "at sequence number %" PRIu64,
+		DBG("Rotate stream %" PRIu64 " at sequence number %" PRIu64,
 				position->stream_id,
 				position->rotate_at_seq_num);
 		ret = lttng_dynamic_buffer_append(&payload, &comm_position,