[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]
[dm-devel] [RFC][PATCH 1/4] dm-log: fix io_client_destroy leak
- From: Takahiro Yasui <tyasui redhat com>
- To: dm-devel redhat com
- Cc: Alasdair G Kergon <agk redhat com>, Masami Hiramatsu <mhiramat redhat com>
- Subject: [dm-devel] [RFC][PATCH 1/4] dm-log: fix io_client_destroy leak
- Date: Tue, 25 Nov 2008 19:01:37 -0500
In create_log_context function, dm_io_client_destroy function needs
to be called, when memory allocation of disk_header, sync_bits and
recovering_bits failed, but dm_io_client_destroy is not called.
This patch fixes it.
Signed-off-by: Takahiro Yasui <tyasui redhat com>
---
drivers/md/dm-log.c | 5 +++++
1 file changed, 5 insertions(+)
Index: linux-2.6.28-rc4/drivers/md/dm-log.c
===================================================================
--- linux-2.6.28-rc4.orig/drivers/md/dm-log.c
+++ linux-2.6.28-rc4/drivers/md/dm-log.c
@@ -467,6 +467,7 @@ static int create_log_context(struct dm_
lc->disk_header = vmalloc(buf_size);
if (!lc->disk_header) {
DMWARN("couldn't allocate disk log buffer");
+ dm_io_client_destroy(lc->io_req.client);
kfree(lc);
return -ENOMEM;
}
@@ -483,6 +484,8 @@ static int create_log_context(struct dm_
if (!dev)
vfree(lc->clean_bits);
vfree(lc->disk_header);
+ if (dev)
+ dm_io_client_destroy(lc->io_req.client);
kfree(lc);
return -ENOMEM;
}
@@ -496,6 +499,8 @@ static int create_log_context(struct dm_
if (!dev)
vfree(lc->clean_bits);
vfree(lc->disk_header);
+ if (dev)
+ dm_io_client_destroy(lc->io_req.client);
kfree(lc);
return -ENOMEM;
}
[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]