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

!5059 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/BEXE3HSFLSJZNTP5J4MVKHL6JXH62EJH/ 
 
https://gitee.com/src-openeuler/kernel/issues/I8YCAG 
 
Link:https://gitee.com/openeuler/kernel/pulls/5059

 

Reviewed-by: default avatarYu Kuai <yukuai3@huawei.com>
Reviewed-by: default avatarZheng Zengkai <zhengzengkai@huawei.com>
Signed-off-by: default avatarZheng Zengkai <zhengzengkai@huawei.com>
parents bba41c23 a5b1dd11

drivers/md/dm-core.h

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



#define DM_RESERVED_MAX_IOS		1024

#define DM_MAX_TARGETS			1048576

#define DM_MAX_TARGET_PARAMS		1024



struct dm_io;

drivers/md/dm-ioctl.c

+2 −1
Original line number Diff line number Diff line
@@ -1952,7 +1952,8 @@ static int copy_params(struct dm_ioctl __user *user, struct dm_ioctl *param_kern 
			   minimum_data_size - sizeof(param_kernel->version)))

		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)) {

		DMERR("Invalid data size in the ioctl structure: %u",

		      param_kernel->data_size);

		return -EINVAL;

drivers/md/dm-table.c

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

		    unsigned int 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;
@@ -144,7 +149,7 @@ int dm_table_create(struct dm_table **result, blk_mode_t mode, 


	if (!num_targets) {

		kfree(t);

		return -ENOMEM;

		return -EOVERFLOW;

	}



	if (alloc_targets(t, num_targets)) {