Unverified Commit 06d20965 authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!5051 dm: limit the number of targets and parameter size area 

Merge Pull Request from: @ci-robot 
 
PR sync from: Li Lingfeng <lilingfeng3@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/YBBJNEV3U3L63SDZPVUNBNP5KKXJ4IRY/ 
 
https://gitee.com/src-openeuler/kernel/issues/I8YCAG 
 
Link:https://gitee.com/openeuler/kernel/pulls/5051

 

Reviewed-by: default avatarJialin Zhang <zhangjialin11@huawei.com>
Reviewed-by: default avatarYu Kuai <yukuai3@huawei.com>
Signed-off-by: default avatarJialin Zhang <zhangjialin11@huawei.com>
parents 9c0b93a0 8d73169f

drivers/md/dm-core.h

+2 −0
Original line number Diff line number Diff line
@@ -19,6 +19,8 @@ 
#include "dm.h"



#define DM_RESERVED_MAX_IOS		1024

#define DM_MAX_TARGETS			1048576

#define DM_MAX_TARGET_PARAMS		1024



struct dm_kobject_holder {

	struct kobject kobj;

drivers/md/dm-ioctl.c

+2 −1
Original line number Diff line number Diff line
@@ -1771,7 +1771,8 @@ static int copy_params(struct dm_ioctl __user *user, struct dm_ioctl *param_kern 
	if (copy_from_user(param_kernel, user, minimum_data_size))

		return -EFAULT;



	if (param_kernel->data_size < minimum_data_size)

	if (unlikely(param_kernel->data_size < minimum_data_size) ||

	    unlikely(param_kernel->data_size > DM_MAX_TARGETS * DM_MAX_TARGET_PARAMS))

		return -EINVAL;



	secure_data = param_kernel->flags & DM_SECURE_DATA_FLAG;

drivers/md/dm-table.c

+7 −2
Original line number Diff line number Diff line
@@ -144,7 +144,12 @@ static int alloc_targets(struct dm_table *t, unsigned int num) 
int dm_table_create(struct dm_table **result, fmode_t mode,

		    unsigned num_targets, struct mapped_device *md)

{

	struct dm_table *t = kzalloc(sizeof(*t), GFP_KERNEL);

	struct dm_table *t;



	if (num_targets > DM_MAX_TARGETS)

		return -EOVERFLOW;



	t = kzalloc(sizeof(*t), GFP_KERNEL);



	if (!t)

		return -ENOMEM;
@@ -158,7 +163,7 @@ int dm_table_create(struct dm_table **result, fmode_t mode, 


	if (!num_targets) {

		kfree(t);

		return -ENOMEM;

		return -EOVERFLOW;

	}



	if (alloc_targets(t, num_targets)) {