summaryrefslogtreecommitdiffstats
path: root/drivers/misc/xilinx_sdfec.c
diff options
context:
space:
mode:
authorLuc Van Oostenryck <luc.vanoostenryck@gmail.com>2019-12-09 22:37:19 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2020-01-14 15:16:51 +0100
commita6b07e89fd8530b943217ccd4909d1bf65acc45d (patch)
tree08719a0ffe01dfdebd13ce06465bfed6d6967bb6 /drivers/misc/xilinx_sdfec.c
parentdrivers/misc: ti-st: remove redundant assignment to variables i and flags (diff)
downloadlinux-a6b07e89fd8530b943217ccd4909d1bf65acc45d.tar.xz
linux-a6b07e89fd8530b943217ccd4909d1bf65acc45d.zip
misc: xilinx_sdfec: add missing __user annotation
The second arg of xsdfec_set_order() is a 'void __user *' and this pointer is then used in get_user() which expect a __user pointer. But get_user() can't be used with a void pointer, it a pointer to the effective type. This is done here by casting the argument to a pointer to the effective type but the __user is missing in the cast. Fix this by adding the missing __user in the cast. CC: Derek Kiernan <derek.kiernan@xilinx.com> CC: Dragan Cvetic <dragan.cvetic@xilinx.com> Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com> Acked-by: Dragan Cvetic <dragan.cvetic@xilinx.com> Link: https://lore.kernel.org/r/20191209213719.58037-1-luc.vanoostenryck@gmail.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/misc/xilinx_sdfec.c')
-rw-r--r--drivers/misc/xilinx_sdfec.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/misc/xilinx_sdfec.c b/drivers/misc/xilinx_sdfec.c
index 11835969e982..f05e1b4c2826 100644
--- a/drivers/misc/xilinx_sdfec.c
+++ b/drivers/misc/xilinx_sdfec.c
@@ -733,7 +733,7 @@ static int xsdfec_set_order(struct xsdfec_dev *xsdfec, void __user *arg)
enum xsdfec_order order;
int err;
- err = get_user(order, (enum xsdfec_order *)arg);
+ err = get_user(order, (enum xsdfec_order __user *)arg);
if (err)
return -EFAULT;