X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=lttng-tracker-id.c;h=327ae3462dadcc7119fca2a22094e2a6831b022e;hb=f59ab559642c10d1188b15ad9f717b009a336454;hp=9a4b98b2ef7ef17ffc834b77cc9305c5205346e6;hpb=b260f6996712ed478b3670df5ebd6387fef00044;p=lttng-modules.git

diff --git a/lttng-tracker-id.c b/lttng-tracker-id.c
index 9a4b98b2..327ae346 100644
--- a/lttng-tracker-id.c
+++ b/lttng-tracker-id.c
@@ -75,6 +75,7 @@ int lttng_id_tracker_add(struct lttng_id_tracker *lf, int id)
 	struct lttng_id_tracker_rcu *p = lf->p;
 	uint32_t hash = hash_32(id, 32);
 	bool allocated = false;
+	int ret;
 
 	if (!p) {
 		p = lttng_id_tracker_rcu_create();
@@ -84,18 +85,28 @@ int lttng_id_tracker_add(struct lttng_id_tracker *lf, int id)
 	}
 	head = &p->id_hash[hash & (LTTNG_ID_TABLE_SIZE - 1)];
 	lttng_hlist_for_each_entry(e, head, hlist) {
-		if (id == e->id)
-			return -EEXIST;
+		if (id == e->id) {
+			ret = -EEXIST;
+			goto error;
+		}
 	}
 	e = kmalloc(sizeof(struct lttng_id_hash_node), GFP_KERNEL);
-	if (!e)
-		return -ENOMEM;
+	if (!e) {
+		ret = -ENOMEM;
+		goto error;
+	}
 	e->id = id;
 	hlist_add_head_rcu(&e->hlist, head);
 	if (allocated) {
 		rcu_assign_pointer(lf->p, p);
 	}
 	return 0;
+
+error:
+	if (allocated) {
+		kfree(p);
+	}
+	return ret;
 }
 
 static