Fix relayd: stream index file created in the wrong directory

[lttng-tools.git] / src / common / index / index.c

diff --git a/src/common/index/index.c b/src/common/index/index.c
index b481badb1d6556e2f6f1807264d33e81e6be41b2..fae1bfd4bcabdd9ba90ee3600733264e581ec7dc 100644 (file)
--- a/src/common/index/index.c
+++ b/src/common/index/index.c
@@ -35,7 +35,7 @@
  *
  * Return allocated struct lttng_index_file, NULL on error.
  */
-struct lttng_index_file *lttng_index_file_create(char *path_name,
+struct lttng_index_file *lttng_index_file_create(const char *path_name,
                char *stream_name, int uid, int gid,
                uint64_t size, uint64_t count, uint32_t major, uint32_t minor)
 {
@@ -96,7 +96,6 @@ struct lttng_index_file *lttng_index_file_create(char *path_name,
        size_ret = lttng_write(fd, &hdr, sizeof(hdr));
        if (size_ret < sizeof(hdr)) {
                PERROR("write index header");
-               ret = -1;
                goto error;
        }
        index_file->fd = fd;
@@ -128,12 +127,16 @@ error:
 int lttng_index_file_write(const struct lttng_index_file *index_file,
                const struct ctf_packet_index *element)
 {
+       int fd;
+       size_t len;
        ssize_t ret;
-       int fd = index_file->fd;
-       size_t len = index_file->element_len;
 
+       assert(index_file);
        assert(element);
 
+       fd = index_file->fd;
+       len = index_file->element_len;
+
        if (fd < 0) {
                goto error;
        }
@@ -219,11 +222,7 @@ struct lttng_index_file *lttng_index_file_open(const char *path_name,
        DBG("Index opening file %s in read only", fullpath);
        read_fd = open(fullpath, O_RDONLY);
        if (read_fd < 0) {
-               if (errno == ENOENT) {
-                       ret = -ENOENT;
-               } else {
-                       PERROR("opening index in read-only");
-               }
+               PERROR("opening index in read-only");
                goto error;
        }
 
@@ -267,7 +266,6 @@ error_close:
                        PERROR("close read fd %d", read_fd);
                }
        }
-       ret = -1;
 
 error:
        free(index_file);