From 12ac7f2baa3691eaf74d468aedb7615f6ddf6d60 Mon Sep 17 00:00:00 2001 From: Frank Zago Date: Thu, 28 Apr 2016 17:07:16 -0400 Subject: [PATCH] LU-8014 hsm: remove invalid kuc_free in coordinator Commit 9091a934fc ("LU-8014 hsm: remove kuc_ispayload") moved the KUC allocation in mdt_hsm_agent_send() and I left a kuc_free in an error path in mdt_coordinator(). The HAL is no longer a KUC and it is freed outside the for loop. That kuc_free is incorrect. Signed-off-by: frank zago Change-Id: If0894d1a8bcbc7960b728e33a34bddb95806acf8 Reviewed-on: http://review.whamcloud.com/19869 Tested-by: Jenkins Reviewed-by: Henri Doreau Tested-by: Maloo Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin --- lustre/mdt/mdt_coordinator.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lustre/mdt/mdt_coordinator.c b/lustre/mdt/mdt_coordinator.c index c02f6a1..caa57ca 100644 --- a/lustre/mdt/mdt_coordinator.c +++ b/lustre/mdt/mdt_coordinator.c @@ -546,7 +546,6 @@ static int mdt_coordinator(void *data) "for cookies vector "LPX64"\n", mdt_obd_name(mdt), sz, hal->hal_compound_id); - kuc_free(hal, hsd.request[i].hal_used_sz); continue; } hai = hai_first(hal); -- 1.8.3.1