X-Git-Url: http://git.efficios.com/?a=blobdiff_plain;f=src%2Fplugins%2Flttng-utils%2Fdebug-info%2Fbin-info.c;h=dcb543151744daf76a5ce6a5b5e6517a80fdc25e;hb=118ae153da2f634b562674b14263a471b4c58de1;hp=31c347ea7088348b32af3efed14d07b12d877f44;hpb=c449a5c3412c92591d6eead8b25e1ea239427801;p=babeltrace.git

diff --git a/src/plugins/lttng-utils/debug-info/bin-info.c b/src/plugins/lttng-utils/debug-info/bin-info.c
index 31c347ea..dcb54315 100644
--- a/src/plugins/lttng-utils/debug-info/bin-info.c
+++ b/src/plugins/lttng-utils/debug-info/bin-info.c
@@ -38,6 +38,7 @@
 #include <inttypes.h>
 #include <libgen.h>
 #include <math.h>
+#include <stdbool.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -163,10 +164,9 @@ int bin_info_set_elf_file(struct bin_info *bin)
 {
 	struct bt_fd_cache_handle *elf_handle = NULL;
 	Elf *elf_file = NULL;
+	int ret;
 
-	if (!bin) {
-		goto error;
-	}
+	BT_ASSERT(bin);
 
 	elf_handle = bt_fd_cache_get_handle(bin->fd_cache, bin->elf_path);
 	if (!elf_handle) {
@@ -178,23 +178,30 @@ int bin_info_set_elf_file(struct bin_info *bin)
 	elf_file = elf_begin(bt_fd_cache_handle_get_fd(bin->elf_handle),
 		ELF_C_READ, NULL);
 	if (!elf_file) {
-		BT_COMP_LOGE("elf_begin failed: %s", elf_errmsg(-1));
+		BT_COMP_LOGE_APPEND_CAUSE(bin->self_comp,
+			"elf_begin failed: %s", elf_errmsg(-1));
 		goto error;
 	}
 
 	bin->elf_file = elf_file;
 
 	if (elf_kind(elf_file) != ELF_K_ELF) {
-		BT_COMP_LOGE("Error: %s is not an ELF object", bin->elf_path);
+		BT_COMP_LOGE_APPEND_CAUSE(bin->self_comp,
+			"Error: %s is not an ELF object", bin->elf_path);
 		goto error;
 	}
 
-	return 0;
+
+	ret = 0;
+	goto end;
 
 error:
 	bt_fd_cache_put_handle(bin->fd_cache, elf_handle);
 	elf_end(elf_file);
-	return -1;
+	ret = -1;
+
+end:
+	return ret;
 }
 
 /**
@@ -293,7 +300,7 @@ end:
  *
  * @returns			1 on if the build id of stored in `bin` matches
  *				the build id of the ondisk file.
- *				0 on if they are different or an error occured.
+ *				0 on if they are different or an error occurred.
  */
 static
 int is_build_id_matching(struct bin_info *bin)
@@ -370,6 +377,9 @@ int bin_info_set_build_id(struct bin_info *bin, uint8_t *build_id,
 		goto error;
 	}
 
+	/* Free any previously set build id. */
+	g_free(bin->build_id);
+
 	/* Set the build id. */
 	bin->build_id = g_new0(uint8_t, build_id_len);
 	if (!bin->build_id) {