X-Git-Url: http://git.efficios.com/?p=babeltrace.git;a=blobdiff_plain;f=converter%2Fbabeltrace-log.c;h=9d87776a8f361ade58a7557215d99fe8a56916fd;hp=f7eb5f768d39c88425f3bff276374f8ba47e9074;hb=a7af71619e2ca1e763d90a8156fe4206c012d8cc;hpb=c5e6c71bad2546b4207e1aede426058062010894

diff --git a/converter/babeltrace-log.c b/converter/babeltrace-log.c
index f7eb5f76..9d87776a 100644
--- a/converter/babeltrace-log.c
+++ b/converter/babeltrace-log.c
@@ -33,6 +33,7 @@
 #include <unistd.h>
 #include <errno.h>
 #include <string.h>
+#include <inttypes.h>
 
 #include <babeltrace/babeltrace-internal.h>
 #include <babeltrace/ctf/types.h>
@@ -53,6 +54,7 @@ static const char metadata_fmt[] =
 "/* CTF 1.8 */\n"
 "typealias integer { size = 8; align = 8; signed = false; } := uint8_t;\n"
 "typealias integer { size = 32; align = 32; signed = false; } := uint32_t;\n"
+"typealias integer { size = 64; align = 64; signed = false; } := uint64_t;\n"
 "\n"
 "trace {\n"
 "	major = %u;\n"			/* major (e.g. 0) */
@@ -67,8 +69,8 @@ static const char metadata_fmt[] =
 "\n"
 "stream {\n"
 "	packet.context := struct {\n"
-"		uint32_t content_size;\n"
-"		uint32_t packet_size;\n"
+"		uint64_t content_size;\n"
+"		uint64_t packet_size;\n"
 "	};\n"
 "%s"					/* Stream event header (opt.) */
 "};\n"
@@ -136,24 +138,24 @@ void write_packet_context(struct ctf_stream_pos *pos)
 
 	/* content_size */
 	ctf_dummy_pos(pos, &dummy);
-	ctf_align_pos(&dummy, sizeof(uint32_t) * CHAR_BIT);
-	ctf_move_pos(&dummy, sizeof(uint32_t) * CHAR_BIT);
+	ctf_align_pos(&dummy, sizeof(uint64_t) * CHAR_BIT);
+	ctf_move_pos(&dummy, sizeof(uint64_t) * CHAR_BIT);
 	assert(!ctf_pos_packet(&dummy));
 	
-	ctf_align_pos(pos, sizeof(uint32_t) * CHAR_BIT);
-	*(uint32_t *) ctf_get_pos_addr(pos) = -1U;	/* Not known yet */
-	pos->content_size_loc = (uint32_t *) ctf_get_pos_addr(pos);
-	ctf_move_pos(pos, sizeof(uint32_t) * CHAR_BIT);
+	ctf_align_pos(pos, sizeof(uint64_t) * CHAR_BIT);
+	*(uint64_t *) ctf_get_pos_addr(pos) = ~0ULL;	/* Not known yet */
+	pos->content_size_loc = (uint64_t *) ctf_get_pos_addr(pos);
+	ctf_move_pos(pos, sizeof(uint64_t) * CHAR_BIT);
 
 	/* packet_size */
 	ctf_dummy_pos(pos, &dummy);
-	ctf_align_pos(&dummy, sizeof(uint32_t) * CHAR_BIT);
-	ctf_move_pos(&dummy, sizeof(uint32_t) * CHAR_BIT);
+	ctf_align_pos(&dummy, sizeof(uint64_t) * CHAR_BIT);
+	ctf_move_pos(&dummy, sizeof(uint64_t) * CHAR_BIT);
 	assert(!ctf_pos_packet(&dummy));
 	
-	ctf_align_pos(pos, sizeof(uint32_t) * CHAR_BIT);
-	*(uint32_t *) ctf_get_pos_addr(pos) = pos->packet_size;
-	ctf_move_pos(pos, sizeof(uint32_t) * CHAR_BIT);
+	ctf_align_pos(pos, sizeof(uint64_t) * CHAR_BIT);
+	*(uint64_t *) ctf_get_pos_addr(pos) = pos->packet_size;
+	ctf_move_pos(pos, sizeof(uint64_t) * CHAR_BIT);
 }
 
 static
@@ -209,7 +211,7 @@ retry:
 		write_packet_header(pos, s_uuid);
 		write_packet_context(pos);
 		if (attempt++ == 1) {
-			fprintf(stderr, "[Error] Line too large for packet size (%zukB) (discarded)\n",
+			fprintf(stderr, "[Error] Line too large for packet size (%" PRIu64 "kB) (discarded)\n",
 				pos->packet_size / CHAR_BIT / 1024);
 			return;
 		}
@@ -229,10 +231,14 @@ void trace_text(FILE *input, int output)
 	ssize_t len;
 	char *line = NULL, *nl;
 	size_t linesize;
+	int ret;
 
 	memset(&pos, 0, sizeof(pos));
-	ctf_init_pos(&pos, output, O_RDWR);
-
+	ret = ctf_init_pos(&pos, output, O_RDWR);
+	if (ret) {
+		fprintf(stderr, "Error in ctf_init_pos\n");
+		return;
+	}
 	write_packet_header(&pos, s_uuid);
 	write_packet_context(&pos);
 	for (;;) {
@@ -247,7 +253,10 @@ void trace_text(FILE *input, int output)
 			trace_string(line, &pos, strlen(line) + 1);
 		}
 	}
-	ctf_fini_pos(&pos);
+	ret = ctf_fini_pos(&pos);
+	if (ret) {
+		fprintf(stderr, "Error in ctf_fini_pos\n");
+	}
 }
 
 static
@@ -345,7 +354,9 @@ int main(int argc, char **argv)
 	print_metadata(metadata_fp);
 	trace_text(stdin, fd);
 
-	close(fd);
+	ret = close(fd);
+	if (ret)
+		perror("close");
 	exit(EXIT_SUCCESS);
 
 	/* error handling */