Commit b0daf15d authored by Boris BREZILLON's avatar Boris BREZILLON Committed by Zefan Li

mtd: mtdpart: fix add_mtd_partitions error path

commit e5bae867 upstream.

If we fail to allocate a partition structure in the middle of the partition
creation process, the already allocated partitions are never removed, which
means they are still present in the partition list and their resources are
never freed.
Signed-off-by: default avatarBoris Brezillon <boris.brezillon@free-electrons.com>
Signed-off-by: default avatarBrian Norris <computersforpeace@gmail.com>
Signed-off-by: default avatarZefan Li <lizefan@huawei.com>
parent e9bceb03
...@@ -632,8 +632,10 @@ int add_mtd_partitions(struct mtd_info *master, ...@@ -632,8 +632,10 @@ int add_mtd_partitions(struct mtd_info *master,
for (i = 0; i < nbparts; i++) { for (i = 0; i < nbparts; i++) {
slave = allocate_partition(master, parts + i, i, cur_offset); slave = allocate_partition(master, parts + i, i, cur_offset);
if (IS_ERR(slave)) if (IS_ERR(slave)) {
del_mtd_partitions(master);
return PTR_ERR(slave); return PTR_ERR(slave);
}
mutex_lock(&mtd_partitions_mutex); mutex_lock(&mtd_partitions_mutex);
list_add(&slave->list, &mtd_partitions); list_add(&slave->list, &mtd_partitions);
......
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