Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1ddc0c3f authored by Mikulas Patocka's avatar Mikulas Patocka Committed by Greg Kroah-Hartman
Browse files

dm: limit the number of targets and parameter size area 



commit bd504bcfec41a503b32054da5472904b404341a4 upstream.

The kvmalloc function fails with a warning if the size is larger than
INT_MAX. The warning was triggered by a syscall testing robot.

In order to avoid the warning, this commit limits the number of targets to
1048576 and the size of the parameter area to 1073741824.

Signed-off-by: default avatarMikulas Patocka <mpatocka@redhat.com>
Signed-off-by: default avatarMike Snitzer <snitzer@kernel.org>
[srish: Apply to stable branch linux-4.19.y]
Signed-off-by: default avatarSrish Srinivasan <srish.srinivasan@broadcom.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent aa62ab6a

drivers/md/dm-core.h

+2 −0
Original line number Diff line number Diff line
@@ -18,6 +18,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
@@ -1734,7 +1734,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
@@ -187,7 +187,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;
@@ -202,7 +207,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)) {