Skip to content

Commit

Permalink
media: v4l2-ioctl: rename a temp var that stores _IOC_SIZE(cmd)
Browse files Browse the repository at this point in the history
Instead of just calling it as "size", let's name it as "ioc_size",
as it reflects better its contents.

As this is constant along the function, also mark it as const.

Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
  • Loading branch information
Mauro Carvalho Chehab committed Mar 26, 2018
1 parent 6ccd228 commit f8a695c
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions drivers/media/v4l2-core/v4l2-ioctl.c
Original file line number Diff line number Diff line change
Expand Up @@ -2833,23 +2833,23 @@ video_usercopy(struct file *file, unsigned int cmd, unsigned long arg,
size_t array_size = 0;
void __user *user_ptr = NULL;
void **kernel_ptr = NULL;
size_t size = _IOC_SIZE(cmd);
const size_t ioc_size = _IOC_SIZE(cmd);

/* Copy arguments into temp kernel buffer */
if (_IOC_DIR(cmd) != _IOC_NONE) {
if (size <= sizeof(sbuf)) {
if (ioc_size <= sizeof(sbuf)) {
parg = sbuf;
} else {
/* too big to allocate from stack */
mbuf = kvmalloc(size, GFP_KERNEL);
mbuf = kvmalloc(ioc_size, GFP_KERNEL);
if (NULL == mbuf)
return -ENOMEM;
parg = mbuf;
}

err = -EFAULT;
if (_IOC_DIR(cmd) & _IOC_WRITE) {
unsigned int n = size;
unsigned int n = ioc_size;

/*
* In some cases, only a few fields are used as input,
Expand All @@ -2870,11 +2870,11 @@ video_usercopy(struct file *file, unsigned int cmd, unsigned long arg,
goto out;

/* zero out anything we don't copy from userspace */
if (n < size)
memset((u8 *)parg + n, 0, size - n);
if (n < ioc_size)
memset((u8 *)parg + n, 0, ioc_size - n);
} else {
/* read-only ioctl */
memset(parg, 0, size);
memset(parg, 0, ioc_size);
}
}

Expand Down Expand Up @@ -2932,7 +2932,7 @@ video_usercopy(struct file *file, unsigned int cmd, unsigned long arg,
switch (_IOC_DIR(cmd)) {
case _IOC_READ:
case (_IOC_WRITE | _IOC_READ):
if (copy_to_user((void __user *)arg, parg, size))
if (copy_to_user((void __user *)arg, parg, ioc_size))
err = -EFAULT;
break;
}
Expand Down

0 comments on commit f8a695c

Please sign in to comment.