X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=src%2Fcommon%2Findex%2Findex.c;h=40ac0de199cd53bdcba43bb80bbd91253e63276e;hb=2a85be8e0e679da996b48252b1d9aebb9bb29126;hp=c94ec701c869f2df3bf61e2845409d3c46697131;hpb=0b0ac4a97d325a6938992f5c04987d5d48167a21;p=lttng-tools.git

diff --git a/src/common/index/index.c b/src/common/index/index.c
index c94ec701c..40ac0de19 100644
--- 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;
@@ -172,10 +171,14 @@ int lttng_index_file_read(const struct lttng_index_file *index_file,
 	}
 
 	ret = lttng_read(fd, element, len);
-	if (ret < len) {
+	if (ret < 0) {
 		PERROR("read index file");
 		goto error;
 	}
+	if (ret < len) {
+		ERR("lttng_read expected %zu, returned %zd", len, ret);
+		goto error;
+	}
 	return 0;
 
 error:
@@ -223,11 +226,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;
 	}
 
@@ -271,7 +270,6 @@ error_close:
 			PERROR("close read fd %d", read_fd);
 		}
 	}
-	ret = -1;
 
 error:
 	free(index_file);