Commit c8897d10 authored by Bart Van Assche's avatar Bart Van Assche Committed by Luis Henriques

target: Fix a memory leak in target_dev_lba_map_store()

commit f0a8afec upstream.

strsep() modifies its first argument. Make the pointer passed to kfree()
match the return value of kmalloc().

Fixes: 229d4f11 (commit "target_core_alua: Referrals configfs integration")
Signed-off-by: default avatarBart Van Assche <bart.vanassche@sandisk.com>
Cc: Hannes Reinecke <hare@suse.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Andy Grover <agrover@redhat.com>
Cc: Sagi Grimberg <sagig@mellanox.com>
Signed-off-by: default avatarNicholas Bellinger <nab@linux-iscsi.org>
Signed-off-by: default avatarLuis Henriques <luis.henriques@canonical.com>
parent f5117c4c
...@@ -1810,14 +1810,14 @@ static ssize_t target_core_store_dev_lba_map( ...@@ -1810,14 +1810,14 @@ static ssize_t target_core_store_dev_lba_map(
struct se_device *dev = p; struct se_device *dev = p;
struct t10_alua_lba_map *lba_map = NULL; struct t10_alua_lba_map *lba_map = NULL;
struct list_head lba_list; struct list_head lba_list;
char *map_entries, *ptr; char *map_entries, *orig, *ptr;
char state; char state;
int pg_num = -1, pg; int pg_num = -1, pg;
int ret = 0, num = 0, pg_id, alua_state; int ret = 0, num = 0, pg_id, alua_state;
unsigned long start_lba = -1, end_lba = -1; unsigned long start_lba = -1, end_lba = -1;
unsigned long segment_size = -1, segment_mult = -1; unsigned long segment_size = -1, segment_mult = -1;
map_entries = kstrdup(page, GFP_KERNEL); orig = map_entries = kstrdup(page, GFP_KERNEL);
if (!map_entries) if (!map_entries)
return -ENOMEM; return -ENOMEM;
...@@ -1915,7 +1915,7 @@ static ssize_t target_core_store_dev_lba_map( ...@@ -1915,7 +1915,7 @@ static ssize_t target_core_store_dev_lba_map(
} else } else
core_alua_set_lba_map(dev, &lba_list, core_alua_set_lba_map(dev, &lba_list,
segment_size, segment_mult); segment_size, segment_mult);
kfree(map_entries); kfree(orig);
return count; return count;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment