[PATCH] missing copy_from_user check in mixcom driver.

9d5a6ebb · Dave Jones · Linus Torvalds · 85b00e35 · 9d5a6ebb
Commit 9d5a6ebb authored Aug 13, 2003 by Dave Jones Committed by Linus Torvalds Aug 13, 2003
Show whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

drivers/net/wan/comx-hw-mixcom.c drivers/net/wan/comx-hw-mixcom.c +4 -1

No files found.
--- a/drivers/net/wan/comx-hw-mixcom.c
+++ b/drivers/net/wan/comx-hw-mixcom.c
@@ -763,7 +763,10 @@ static int mixcom_write_proc(struct file *file, const char *buffer,
 		return -ENOMEM;
 	}
-	copy_from_user(page, buffer, count = min_t(unsigned long, count, PAGE_SIZE));
+	if (copy_from_user(page, buffer, count = min_t(unsigned long, count, PAGE_SIZE))) {
+		free_page((unsigned long)page);
+		return -EFAULT;
+	}
 	if (*(page + count - 1) == '\n') {
 		*(page + count - 1) = 0;
 	}