[PATCH v1.1 1/1] videodev2: Mark all user pointers as such

* [PATCH v1.1 1/1] videodev2: Mark all user pointers as such
@ 2018-04-12 13:03 Sakari Ailus
  2018-04-12 13:43 ` Sakari Ailus
  2018-04-13 20:03 ` kbuild test robot
  0 siblings, 2 replies; 3+ messages in thread
From: Sakari Ailus @ 2018-04-12 13:03 UTC (permalink / raw)
  To: linux-media; +Cc: mchehab, hverkuil

A number of uAPI structs have pointers but some lack the __user modifier.
Add this to the pointers that do not have it.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
---
since v1:

- Against the master branch (not Hans's reqv10)

 include/uapi/linux/videodev2.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h
index 600877be5c22..ac9c65ee4c56 100644
--- a/include/uapi/linux/videodev2.h
+++ b/include/uapi/linux/videodev2.h
@@ -929,7 +929,7 @@ struct v4l2_buffer {
 	union {
 		__u32           offset;
 		unsigned long   userptr;
-		struct v4l2_plane *planes;
+		struct v4l2_plane __user *planes;
 		__s32		fd;
 	} m;
 	__u32			length;
@@ -1006,7 +1006,7 @@ struct v4l2_framebuffer {
 	__u32			flags;
 /* FIXME: in theory we should pass something like PCI device + memory
  * region + offset instead of some physical address */
-	void                    *base;
+	void __user		*base;
 	struct {
 		__u32		width;
 		__u32		height;
@@ -1593,7 +1593,7 @@ struct v4l2_ext_controls {
 	__u32 count;
 	__u32 error_idx;
 	__u32 reserved[2];
-	struct v4l2_ext_control *controls;
+	struct v4l2_ext_control __user *controls;
 };
 
 #define V4L2_CTRL_ID_MASK	  (0x0fffffff)
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread