[AGPGART] sanitise agp mode input on setup ioctl.

9c6c0b9e · Dave Jones · Dave Jones · 1e464de0 · 9c6c0b9e
Commit 9c6c0b9e authored Mar 28, 2003 by Dave Jones Committed by Dave Jones Mar 28, 2003
Hide whitespace changes
Inline Side-by-side

Showing with 11 additions and 3 deletions

drivers/char/agp/frontend.c drivers/char/agp/frontend.c +11 -3

No files found.
--- a/drivers/char/agp/frontend.c
+++ b/drivers/char/agp/frontend.c
@@ -805,11 +805,19 @@ static int agpioc_setup_wrap(agp_file_private * priv, unsigned long arg)
 {
 	agp_setup mode;
-	if (copy_from_user(&mode, (void *) arg, sizeof(agp_setup))) {
+	if (copy_from_user(&mode, (void *) arg, sizeof(agp_setup)))
 		return -EFAULT;
+	switch (mode.agp_mode) {
+		case 1:
+		case 2:
+		case 4:
+		case 8:
+			agp_enable(mode.agp_mode);
+			return 0;
+		default:;
 	}
-	agp_enable(mode.agp_mode);
+	return -EINVAL;
-	return 0;
 }
 static int agpioc_reserve_wrap(agp_file_private * priv, unsigned long arg)