Patch "media: v4l2-compat-ioctl32.c: fix the indentation" has been added to the 3.18-stable tree
Subsystems (2)
This is a note to let you know that I've just added the patch titled
media: v4l2-compat-ioctl32.c: fix the indentation
to the 3.18-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
media-v4l2-compat-ioctl32.c-fix-the-indentation.patch
and it can be found in the queue-3.18 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.
Quoted 1 line
From
b7b957d429f601d6d1942122b339474f31191d75Mon Sep 17 00:00:00 2001
From: Hans Verkuil <[email protected]>
Date: Wed, 24 Jan 2018 04:35:48 -0500
Subject: media: v4l2-compat-ioctl32.c: fix the indentation
From: Hans Verkuil <[email protected]>
commit b7b957d429f601d6d1942122b339474f31191d75 upstream.
The indentation of this source is all over the place. Fix this. This patch only changes whitespace.
--- drivers/media/v4l2-core/v4l2-compat-ioctl32.c | 216 +++++++++++++------------- 1 file changed, 108 insertions(+), 108 deletions(-)
drivers/media/v4l2-core/v4l2-compat-ioctl32.c +108 -108
@@ -48,11 +48,11 @@ struct v4l2_window32 {static int get_v4l2_window32 (struct v4l2_window *kp,struct v4l2_window32 __user *up) {if (!access_ok (VERIFY_READ , up,sizeof (struct v4l2_window32 )) || -copy_from_user (&kp->w, &up->w,sizeof (up->w)) || -get_user (kp->field, &up->field) || -get_user (kp->chromakey, &up->chromakey) || -get_user (kp->clipcount, &up->clipcount)) -return -EFAULT ; +copy_from_user (&kp->w, &up->w,sizeof (up->w)) || +get_user (kp->field, &up->field) || +get_user (kp->chromakey, &up->chromakey) || +get_user (kp->clipcount, &up->clipcount)) +return -EFAULT ;if (kp->clipcount >2048 )return -EINVAL ;if (kp->clipcount) {
@@ -82,10 +82,10 @@ static int get_v4l2_window32(struct v4l2static int put_v4l2_window32 (struct v4l2_window *kp,struct v4l2_window32 __user *up) {if (copy_to_user (&up->w, &kp->w,sizeof (kp->w)) || -put_user (kp->field, &up->field) || -put_user (kp->chromakey, &up->chromakey) || -put_user (kp->clipcount, &up->clipcount)) -return -EFAULT ; +put_user (kp->field, &up->field) || +put_user (kp->chromakey, &up->chromakey) || +put_user (kp->clipcount, &up->clipcount)) +return -EFAULT ;return 0 ; }
@@ -97,7 +97,7 @@ static inline int get_v4l2_pix_format(st }static inline int get_v4l2_pix_format_mplane (struct v4l2_pix_format_mplane *kp, -struct v4l2_pix_format_mplane __user *up) +struct v4l2_pix_format_mplane __user *up) {if (copy_from_user (kp, up,sizeof (struct v4l2_pix_format_mplane )))return -EFAULT ;
@@ -112,7 +112,7 @@ static inline int put_v4l2_pix_format(st }static inline int put_v4l2_pix_format_mplane (struct v4l2_pix_format_mplane *kp, -struct v4l2_pix_format_mplane __user *up) +struct v4l2_pix_format_mplane __user *up) {if (copy_to_user (up, kp,sizeof (struct v4l2_pix_format_mplane )))return -EFAULT ;
@@ -200,7 +200,7 @@ static int __get_v4l2_format32(struct v4return get_v4l2_sliced_vbi_format (&kp->fmt.sliced, &up->fmt.sliced);default :printk (KERN_INFO "compat_ioctl32: unexpected VIDIOC_FMT type %d\n" , - kp->type); + kp->type);return -EINVAL ; } }
@@ -241,7 +241,7 @@ static int __put_v4l2_format32(struct v4return put_v4l2_sliced_vbi_format (&kp->fmt.sliced, &up->fmt.sliced);default :printk (KERN_INFO "compat_ioctl32: unexpected VIDIOC_FMT type %d\n" , - kp->type); + kp->type);return -EINVAL ; } }
@@ -275,7 +275,7 @@ static int get_v4l2_standard32(struct v4 {/* other fields are not set by the user, nor used by the driver */ if (!access_ok (VERIFY_READ , up,sizeof (struct v4l2_standard32 )) || -get_user (kp->index, &up->index)) +get_user (kp->index, &up->index))return -EFAULT ;return 0 ; }
@@ -283,13 +283,13 @@ static int get_v4l2_standard32(struct v4static int put_v4l2_standard32 (struct v4l2_standard *kp,struct v4l2_standard32 __user *up) {if (!access_ok (VERIFY_WRITE , up,sizeof (struct v4l2_standard32 )) || -put_user (kp->index, &up->index) || -copy_to_user (up->id, &kp->id,sizeof (__u64)) || -copy_to_user (up->name, kp->name,24 ) || -copy_to_user (&up->frameperiod, &kp->frameperiod,sizeof (kp->frameperiod)) || -put_user (kp->framelines, &up->framelines) || -copy_to_user (up->reserved, kp->reserved,4 *sizeof (__u32))) -return -EFAULT ; +put_user (kp->index, &up->index) || +copy_to_user (up->id, &kp->id,sizeof (__u64)) || +copy_to_user (up->name, kp->name,24 ) || +copy_to_user (&up->frameperiod, &kp->frameperiod,sizeof (kp->frameperiod)) || +put_user (kp->framelines, &up->framelines) || +copy_to_user (up->reserved, kp->reserved,4 *sizeof (__u32))) +return -EFAULT ;return 0 ; }
@@ -329,14 +329,14 @@ struct v4l2_buffer32 { };static int get_v4l2_plane32 (struct v4l2_plane __user *up,struct v4l2_plane32 __user *up32, -enum v4l2_memory memory) +enum v4l2_memory memory) {void __user *up_pln;compat_long_t p;if (copy_in_user (up, up32,2 *sizeof (__u32)) || -copy_in_user (&up->data_offset, &up32->data_offset, -sizeof (__u32))) +copy_in_user (&up->data_offset, &up32->data_offset, +sizeof (__u32)))return -EFAULT ;if (memory ==V4L2_MEMORY_USERPTR ) {
@@ -350,7 +350,7 @@ static int get_v4l2_plane32(struct v4l2_return -EFAULT ; }else {if (copy_in_user (&up->m.mem_offset, &up32->m.mem_offset, -sizeof (__u32))) +sizeof (__u32)))return -EFAULT ; }
@@ -358,23 +358,23 @@ static int get_v4l2_plane32(struct v4l2_ }static int put_v4l2_plane32 (struct v4l2_plane __user *up,struct v4l2_plane32 __user *up32, -enum v4l2_memory memory) +enum v4l2_memory memory) {if (copy_in_user (up32, up,2 *sizeof (__u32)) || -copy_in_user (&up32->data_offset, &up->data_offset, -sizeof (__u32))) +copy_in_user (&up32->data_offset, &up->data_offset, +sizeof (__u32)))return -EFAULT ;/* For MMAP, driver might've set up the offset, so copy it back. * USERPTR stays the same (was userspace-provided), so no copying. */ if (memory ==V4L2_MEMORY_MMAP )if (copy_in_user (&up32->m.mem_offset, &up->m.mem_offset, -sizeof (__u32))) +sizeof (__u32)))return -EFAULT ;/* For DMABUF, driver might've set up the fd, so copy it back. */ if (memory ==V4L2_MEMORY_DMABUF )if (copy_in_user (&up32->m.fd, &up->m.fd, -sizeof (int ))) +sizeof (int )))return -EFAULT ;return 0 ;
@@ -389,19 +389,19 @@ static int get_v4l2_buffer32(struct v4l2int ret;if (!access_ok (VERIFY_READ , up,sizeof (struct v4l2_buffer32 )) || -get_user (kp->index, &up->index) || -get_user (kp->type, &up->type) || -get_user (kp->flags, &up->flags) || -get_user (kp->memory, &up->memory) || -get_user (kp->length, &up->length)) -return -EFAULT ; +get_user (kp->index, &up->index) || +get_user (kp->type, &up->type) || +get_user (kp->flags, &up->flags) || +get_user (kp->memory, &up->memory) || +get_user (kp->length, &up->length)) +return -EFAULT ;if (V4L2_TYPE_IS_OUTPUT (kp->type))if (get_user (kp->bytesused, &up->bytesused) || -get_user (kp->field, &up->field) || -get_user (kp->timestamp.tv_sec, &up->timestamp.tv_sec) || -get_user (kp->timestamp.tv_usec, - &up->timestamp.tv_usec)) +get_user (kp->field, &up->field) || +get_user (kp->timestamp.tv_sec, &up->timestamp.tv_sec) || +get_user (kp->timestamp.tv_usec, + &up->timestamp.tv_usec))return -EFAULT ;if (V4L2_TYPE_IS_MULTIPLANAR (kp->type)) {
@@ -418,13 +418,13 @@ static int get_v4l2_buffer32(struct v4l2 uplane32 =compat_ptr (p);if (!access_ok (VERIFY_READ , uplane32, - num_planes *sizeof (struct v4l2_plane32 ))) + num_planes *sizeof (struct v4l2_plane32 )))return -EFAULT ;/* We don't really care if userspace decides to kill itself * by passing a very big num_planes value */ uplane =compat_alloc_user_space (num_planes * -sizeof (struct v4l2_plane )); +sizeof (struct v4l2_plane )); kp->m.planes = (__force struct v4l2_plane *)uplane;while (--num_planes >=0 ) {
@@ -442,12 +442,12 @@ static int get_v4l2_buffer32(struct v4l2break ;case V4L2_MEMORY_USERPTR : { -compat_long_t tmp; +compat_long_t tmp; -if (get_user (tmp, &up->m.userptr)) -return -EFAULT ; +if (get_user (tmp, &up->m.userptr)) +return -EFAULT ; - kp->m.userptr = (unsigned long )compat_ptr (tmp); + kp->m.userptr = (unsigned long )compat_ptr (tmp); }break ;case V4L2_MEMORY_OVERLAY :
@@ -473,22 +473,22 @@ static int put_v4l2_buffer32(struct v4l2int ret;if (!access_ok (VERIFY_WRITE , up,sizeof (struct v4l2_buffer32 )) || -put_user (kp->index, &up->index) || -put_user (kp->type, &up->type) || -put_user (kp->flags, &up->flags) || -put_user (kp->memory, &up->memory)) -return -EFAULT ; +put_user (kp->index, &up->index) || +put_user (kp->type, &up->type) || +put_user (kp->flags, &up->flags) || +put_user (kp->memory, &up->memory)) +return -EFAULT ;if (put_user (kp->bytesused, &up->bytesused) || -put_user (kp->field, &up->field) || -put_user (kp->timestamp.tv_sec, &up->timestamp.tv_sec) || -put_user (kp->timestamp.tv_usec, &up->timestamp.tv_usec) || -copy_to_user (&up->timecode, &kp->timecode,sizeof (struct v4l2_timecode )) || -put_user (kp->sequence, &up->sequence) || -put_user (kp->reserved2, &up->reserved2) || -put_user (kp->reserved, &up->reserved) || -put_user (kp->length, &up->length)) -return -EFAULT ; +put_user (kp->field, &up->field) || +put_user (kp->timestamp.tv_sec, &up->timestamp.tv_sec) || +put_user (kp->timestamp.tv_usec, &up->timestamp.tv_usec) || +copy_to_user (&up->timecode, &kp->timecode,sizeof (struct v4l2_timecode )) || +put_user (kp->sequence, &up->sequence) || +put_user (kp->reserved2, &up->reserved2) || +put_user (kp->reserved, &up->reserved) || +put_user (kp->length, &up->length)) +return -EFAULT ;if (V4L2_TYPE_IS_MULTIPLANAR (kp->type)) { num_planes = kp->length;
@@ -552,11 +552,11 @@ static int get_v4l2_framebuffer32(structu32 tmp;if (!access_ok (VERIFY_READ , up,sizeof (struct v4l2_framebuffer32 )) || -get_user (tmp, &up->base) || -get_user (kp->capability, &up->capability) || -get_user (kp->flags, &up->flags) || -copy_from_user (&kp->fmt, &up->fmt,sizeof (up->fmt))) -return -EFAULT ; +get_user (tmp, &up->base) || +get_user (kp->capability, &up->capability) || +get_user (kp->flags, &up->flags) || +copy_from_user (&kp->fmt, &up->fmt,sizeof (up->fmt))) +return -EFAULT ; kp->base = (__force void *)compat_ptr (tmp);return 0 ; }
@@ -566,11 +566,11 @@ static int put_v4l2_framebuffer32(structu32 tmp = (u32)((unsigned long )kp->base);if (!access_ok (VERIFY_WRITE , up,sizeof (struct v4l2_framebuffer32 )) || -put_user (tmp, &up->base) || -put_user (kp->capability, &up->capability) || -put_user (kp->flags, &up->flags) || -copy_to_user (&up->fmt, &kp->fmt,sizeof (up->fmt))) -return -EFAULT ; +put_user (tmp, &up->base) || +put_user (kp->capability, &up->capability) || +put_user (kp->flags, &up->flags) || +copy_to_user (&up->fmt, &kp->fmt,sizeof (up->fmt))) +return -EFAULT ;return 0 ; }
@@ -602,11 +602,11 @@ static inline int put_v4l2_input32(struc }struct v4l2_ext_controls32 { -__u32 ctrl_class; -__u32 count; -__u32 error_idx; -__u32 reserved[2 ]; -compat_caddr_t controls;/* actually struct v4l2_ext_control32 * */ +__u32 ctrl_class; +__u32 count; +__u32 error_idx; +__u32 reserved[2 ]; +compat_caddr_t controls;/* actually struct v4l2_ext_control32 * */ };struct v4l2_ext_control32 {
@@ -645,11 +645,11 @@ static int get_v4l2_ext_controls32(struccompat_caddr_t p;if (!access_ok (VERIFY_READ , up,sizeof (struct v4l2_ext_controls32 )) || -get_user (kp->ctrl_class, &up->ctrl_class) || -get_user (kp->count, &up->count) || -get_user (kp->error_idx, &up->error_idx) || -copy_from_user (kp->reserved, up->reserved,sizeof (kp->reserved))) -return -EFAULT ; +get_user (kp->ctrl_class, &up->ctrl_class) || +get_user (kp->count, &up->count) || +get_user (kp->error_idx, &up->error_idx) || +copy_from_user (kp->reserved, up->reserved,sizeof (kp->reserved))) +return -EFAULT ; n = kp->count;if (n ==0 ) { kp->controls =NULL ;
@@ -659,7 +659,7 @@ static int get_v4l2_ext_controls32(strucreturn -EFAULT ; ucontrols =compat_ptr (p);if (!access_ok (VERIFY_READ , ucontrols, - n *sizeof (struct v4l2_ext_control32 ))) + n *sizeof (struct v4l2_ext_control32 )))return -EFAULT ; kcontrols =compat_alloc_user_space (n *sizeof (struct v4l2_ext_control )); kp->controls = (__force struct v4l2_ext_control *)kcontrols;
@@ -694,11 +694,11 @@ static int put_v4l2_ext_controls32(struccompat_caddr_t p;if (!access_ok (VERIFY_WRITE , up,sizeof (struct v4l2_ext_controls32 )) || -put_user (kp->ctrl_class, &up->ctrl_class) || -put_user (kp->count, &up->count) || -put_user (kp->error_idx, &up->error_idx) || -copy_to_user (up->reserved, kp->reserved,sizeof (up->reserved))) -return -EFAULT ; +put_user (kp->ctrl_class, &up->ctrl_class) || +put_user (kp->count, &up->count) || +put_user (kp->error_idx, &up->error_idx) || +copy_to_user (up->reserved, kp->reserved,sizeof (up->reserved))) +return -EFAULT ;if (!kp->count)return 0 ;
@@ -706,7 +706,7 @@ static int put_v4l2_ext_controls32(strucreturn -EFAULT ; ucontrols =compat_ptr (p);if (!access_ok (VERIFY_WRITE , ucontrols, - n *sizeof (struct v4l2_ext_control32 ))) + n *sizeof (struct v4l2_ext_control32 )))return -EFAULT ;while (--n >=0 ) {
@@ -743,15 +743,15 @@ struct v4l2_event32 {static int put_v4l2_event32 (struct v4l2_event *kp,struct v4l2_event32 __user *up) {if (!access_ok (VERIFY_WRITE , up,sizeof (struct v4l2_event32 )) || -put_user (kp->type, &up->type) || -copy_to_user (&up->u, &kp->u,sizeof (kp->u)) || -put_user (kp->pending, &up->pending) || -put_user (kp->sequence, &up->sequence) || -put_user (kp->timestamp.tv_sec, &up->timestamp.tv_sec) || -put_user (kp->timestamp.tv_nsec, &up->timestamp.tv_nsec) || -put_user (kp->id, &up->id) || -copy_to_user (up->reserved, kp->reserved,8 *sizeof (__u32))) -return -EFAULT ; +put_user (kp->type, &up->type) || +copy_to_user (&up->u, &kp->u,sizeof (kp->u)) || +put_user (kp->pending, &up->pending) || +put_user (kp->sequence, &up->sequence) || +put_user (kp->timestamp.tv_sec, &up->timestamp.tv_sec) || +put_user (kp->timestamp.tv_nsec, &up->timestamp.tv_nsec) || +put_user (kp->id, &up->id) || +copy_to_user (up->reserved, kp->reserved,8 *sizeof (__u32))) +return -EFAULT ;return 0 ; }
@@ -768,12 +768,12 @@ static int get_v4l2_edid32(struct v4l2_eu32 tmp;if (!access_ok (VERIFY_READ , up,sizeof (struct v4l2_edid32 )) || -get_user (kp->pad, &up->pad) || -get_user (kp->start_block, &up->start_block) || -get_user (kp->blocks, &up->blocks) || -get_user (tmp, &up->edid) || -copy_from_user (kp->reserved, up->reserved,sizeof (kp->reserved))) -return -EFAULT ; +get_user (kp->pad, &up->pad) || +get_user (kp->start_block, &up->start_block) || +get_user (kp->blocks, &up->blocks) || +get_user (tmp, &up->edid) || +copy_from_user (kp->reserved, up->reserved,sizeof (kp->reserved))) +return -EFAULT ; kp->edid = (__force u8 *)compat_ptr (tmp);return 0 ; }
@@ -783,12 +783,12 @@ static int put_v4l2_edid32(struct v4l2_eu32 tmp = (u32)((unsigned long )kp->edid);if (!access_ok (VERIFY_WRITE , up,sizeof (struct v4l2_edid32 )) || -put_user (kp->pad, &up->pad) || -put_user (kp->start_block, &up->start_block) || -put_user (kp->blocks, &up->blocks) || -put_user (tmp, &up->edid) || -copy_to_user (up->reserved, kp->reserved,sizeof (up->reserved))) -return -EFAULT ; +put_user (kp->pad, &up->pad) || +put_user (kp->start_block, &up->start_block) || +put_user (kp->blocks, &up->blocks) || +put_user (tmp, &up->edid) || +copy_to_user (up->reserved, kp->reserved,sizeof (up->reserved))) +return -EFAULT ;return 0 ; }
Patches currently in stable-queue which might be from [email protected] are
queue-3.18/media-v4l2-compat-ioctl32.c-copy-m.userptr-in-put_v4l2_plane32.patch queue-3.18/media-v4l2-compat-ioctl32.c-avoid-sizeof-type.patch queue-3.18/media-v4l2-compat-ioctl32.c-drop-pr_info-for-unknown-buffer-type.patch queue-3.18/media-v4l2-compat-ioctl32.c-add-missing-vidioc_prepare_buf.patch queue-3.18/vb2-v4l2_buf_flag_done-is-set-after-dqbuf.patch queue-3.18/media-v4l2-compat-ioctl32.c-refactor-compat-ioctl32-logic.patch queue-3.18/media-v4l2-ctrls-fix-sparse-warning.patch queue-3.18/media-v4l2-compat-ioctl32.c-fix-ctrl_is_pointer.patch queue-3.18/media-v4l2-compat-ioctl32.c-move-helper-functions-to-__get-put_v4l2_format32.patch queue-3.18/media-media-v4l2-ctrls-volatiles-should-not-generate-ch_value.patch queue-3.18/media-v4l2-compat-ioctl32.c-don-t-copy-back-the-result-for-certain-errors.patch queue-3.18/media-v4l2-compat-ioctl32.c-make-ctrl_is_pointer-work-for-subdevs.patch queue-3.18/media-v4l2-compat-ioctl32.c-fix-the-indentation.patch queue-3.18/media-v4l2-compat-ioctl32-copy-v4l2_window-global_alpha.patch queue-3.18/media-v4l2-ioctl.c-don-t-copy-back-the-result-for-enotty.patch queue-3.18/media-v4l2-compat-ioctl32.c-copy-clip-list-in-put_v4l2_window32.patch