From e9396abf4199c5877026677c344d25555371fad2 Mon Sep 17 00:00:00 2001 From: Hongchao Zhang Date: Thu, 12 Jan 2012 21:29:00 +0800 Subject: [PATCH] LU-980 llog: cleanup return value in llog_client_create in llog_client_create, the newly allocated llog_handle is return by parameter res, but it doesn't be cleaned up if the following operations failed and the corresponding llog_handle is already freed. Signed-off-by: Hongchao Zhang Change-Id: I8b59dfde91da2c20881f29e8ff46a0a93f0ee1b2 Reviewed-on: http://review.whamcloud.com/1958 Tested-by: Hudson Tested-by: Maloo Reviewed-by: Fan Yong Reviewed-by: Bobi Jam Reviewed-by: Oleg Drokin --- lustre/ptlrpc/llog_client.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lustre/ptlrpc/llog_client.c b/lustre/ptlrpc/llog_client.c index 4305375..0a5bb91 100644 --- a/lustre/ptlrpc/llog_client.c +++ b/lustre/ptlrpc/llog_client.c @@ -147,6 +147,7 @@ out: ptlrpc_req_finished(req); return rc; err_free: + *res = NULL; llog_free_handle(handle); goto out; } -- 1.8.3.1