diff options
| author | Geert Uytterhoeven <geert@linux-m68k.org> | 2022-07-11 18:50:30 +0300 |
|---|---|---|
| committer | Helge Deller <deller@gmx.de> | 2022-07-18 08:56:18 +0300 |
| commit | 3ee5e2280343472de16a9048d6b9049c66d572e3 (patch) | |
| tree | 199a9dc7a1f0f05b24f267243b22e6f6fd188e60 /drivers/video/fbdev | |
| parent | 4a13bcd8ccd5cc063969620e68f068a83cccc55a (diff) | |
| download | linux-3ee5e2280343472de16a9048d6b9049c66d572e3.tar.xz | |
video: fbdev: atari: Remove unneeded casts to void *
Arbitrary pointers can be passed to functions accepting "void *" without
casting.
Remove the casts, as they make it impossible to validate types.
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Tested-by: Michael Schmitz <schmitzmic@gmail.com>
Signed-off-by: Helge Deller <deller@gmx.de>
Diffstat (limited to 'drivers/video/fbdev')
| -rw-r--r-- | drivers/video/fbdev/atafb.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/video/fbdev/atafb.c b/drivers/video/fbdev/atafb.c index f20535ea3e54..fbc333d5615d 100644 --- a/drivers/video/fbdev/atafb.c +++ b/drivers/video/fbdev/atafb.c @@ -2599,14 +2599,14 @@ atafb_ioctl(struct fb_info *info, unsigned int cmd, unsigned long arg) switch (cmd) { #ifdef FBCMD_GET_CURRENTPAR case FBCMD_GET_CURRENTPAR: - if (copy_to_user((void *)arg, (void *)¤t_par, + if (copy_to_user((void *)arg, ¤t_par, sizeof(struct atafb_par))) return -EFAULT; return 0; #endif #ifdef FBCMD_SET_CURRENTPAR case FBCMD_SET_CURRENTPAR: - if (copy_from_user((void *)¤t_par, (void *)arg, + if (copy_from_user(¤t_par, (void *)arg, sizeof(struct atafb_par))) return -EFAULT; ata_set_par(¤t_par); |