summaryrefslogtreecommitdiff
path: root/sys/dev/dm/dm_target_stripe.c
diff options
context:
space:
mode:
authoragc <agc@NetBSD.org>2014-08-18 17:16:19 +0000
committeragc <agc@NetBSD.org>2014-08-18 17:16:19 +0000
commitb0cd2b186b069d41c213ec6f76e75d0bc07efc82 (patch)
treebd053f68e69cbf91bf916c3ed5ab67d3b98c590f /sys/dev/dm/dm_target_stripe.c
parenta4f89ee965930d022c3381918c9a1b2f6f08113b (diff)
Avoid a memory leak - from maxv
Diffstat (limited to 'sys/dev/dm/dm_target_stripe.c')
-rw-r--r--sys/dev/dm/dm_target_stripe.c11
1 files changed, 8 insertions, 3 deletions
diff --git a/sys/dev/dm/dm_target_stripe.c b/sys/dev/dm/dm_target_stripe.c
index de5352a8d01..7bbca169961 100644
--- a/sys/dev/dm/dm_target_stripe.c
+++ b/sys/dev/dm/dm_target_stripe.c
@@ -1,4 +1,4 @@
-/*$NetBSD: dm_target_stripe.c,v 1.19 2014/06/14 07:39:00 hannken Exp $*/
+/*$NetBSD: dm_target_stripe.c,v 1.20 2014/08/18 17:16:42 agc Exp $*/
/*
* Copyright (c) 2009 The NetBSD Foundation, Inc.
@@ -156,8 +156,11 @@ dm_target_stripe_init(dm_dev_t * dmv, void **target_config, char *params)
argv[strpi], argv[strpi+1]);
tlc = kmem_alloc(sizeof(*tlc), KM_NOSLEEP);
- if ((tlc->pdev = dm_pdev_insert(argv[strpi])) == NULL)
+ if ((tlc->pdev = dm_pdev_insert(argv[strpi])) == NULL) {
+ kmem_free(tsc, sizeof(*tsc));
+ kmem_free(tlc, sizeof(*tlc));
return ENOENT;
+ }
tlc->offset = atoi(argv[strpi+1]);
/* Insert striping device to linked list. */
@@ -183,8 +186,10 @@ dm_target_stripe_status(void *target_config)
if ((params = kmem_alloc(DM_MAX_PARAMS_SIZE, KM_SLEEP)) == NULL)
return NULL;
- if ((tmp = kmem_alloc(DM_MAX_PARAMS_SIZE, KM_SLEEP)) == NULL)
+ if ((tmp = kmem_alloc(DM_MAX_PARAMS_SIZE, KM_SLEEP)) == NULL) {
+ kmem_free(params, DM_MAX_PARAMS_SIZE);
return NULL;
+ }
snprintf(params, DM_MAX_PARAMS_SIZE, "%d %" PRIu64,
tsc->stripe_num, tsc->stripe_chunksize);