Commit 3ee5e228 authored by Geert Uytterhoeven's avatar Geert Uytterhoeven Committed by Helge Deller
Browse files

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: default avatarGeert Uytterhoeven <geert@linux-m68k.org>
Tested-by: default avatarMichael Schmitz <schmitzmic@gmail.com>
Signed-off-by: default avatarHelge Deller <deller@gmx.de>
parent 4a13bcd8

drivers/video/fbdev/atafb.c

+2 −2
Original line number Diff line number Diff line
@@ -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 *)&current_par,

		if (copy_to_user((void *)arg, &current_par,

				 sizeof(struct atafb_par)))

			return -EFAULT;

		return 0;

#endif

#ifdef FBCMD_SET_CURRENTPAR

	case FBCMD_SET_CURRENTPAR:

		if (copy_from_user((void *)&current_par, (void *)arg,

		if (copy_from_user(&current_par, (void *)arg,

				   sizeof(struct atafb_par)))

			return -EFAULT;

		ata_set_par(&current_par);