From 7c8cb1b6fdd78c3785fa59fd377cd1c316f1fd18 Mon Sep 17 00:00:00 2001 From: Ziye Yang Date: Wed, 31 Jul 2019 21:59:38 +0800 Subject: [PATCH] sock: Add the code to free the entry Checked code, we do not free the memory allocated spdk_sock_placement_id_entry. Signed-off-by: Ziye Yang Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/463725 (master) (cherry picked from commit a2dcdde2da42c8231c1029094beaa76a02ae16ef) Change-Id: Ie614dc17334b21b8904b16ee7e6e68a24e29d6a3 Signed-off-by: Tomasz Zawadzki Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/467126 Tested-by: SPDK CI Jenkins Reviewed-by: Jim Harris Reviewed-by: Ben Walker --- lib/sock/sock.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/sock/sock.c b/lib/sock/sock.c index 9e0f56cb1..29426b1b9 100644 --- a/lib/sock/sock.c +++ b/lib/sock/sock.c @@ -103,6 +103,7 @@ spdk_sock_map_release(int placement_id) entry->ref--; if (!entry->ref) { STAILQ_REMOVE(&g_placement_id_map, entry, spdk_sock_placement_id_entry, link); + free(entry); } break; } @@ -139,6 +140,7 @@ spdk_sock_remove_sock_group_from_map_table(struct spdk_sock_group *group) STAILQ_FOREACH_SAFE(entry, &g_placement_id_map, link, tmp) { if (entry->group == group) { STAILQ_REMOVE(&g_placement_id_map, entry, spdk_sock_placement_id_entry, link); + free(entry); } } pthread_mutex_unlock(&g_map_table_mutex);