From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: Wed, 18 May 2016 18:04:10 +0000 (-0400)
Subject: Fix: bad file descriptors on close after rotation error
X-Git-Tag: v2.7.3~29
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=61921f598074c2b22423b5451dd1b6544379fde1;p=lttng-tools.git

Fix: bad file descriptors on close after rotation error

Ensure we don't try to close output stream file descriptors twice when a
trace file rotation error occurs (once at tracefile rotation, once when
closing the stream). Set the fd value to -1 after the first close to
ensure we don't try to close it again.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
---

diff --git a/src/common/utils.c b/src/common/utils.c
index 217d8c65d..2473ca39e 100644
--- a/src/common/utils.c
+++ b/src/common/utils.c
@@ -836,6 +836,7 @@ int utils_rotate_stream_file(char *path_name, char *file_name, uint64_t size,
 		PERROR("Closing tracefile");
 		goto error;
 	}
+	*stream_fd = -1;
 
 	if (count > 0) {
 		/*