[PATCH] missing copy_from_user check in comx-proto-fr driver

9739f37d · Dave Jones · Linus Torvalds · 7bb3f4d2 · 9739f37d
Commit 9739f37d authored Aug 13, 2003 by Dave Jones Committed by Linus Torvalds Aug 13, 2003
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

drivers/net/wan/comx-proto-fr.c drivers/net/wan/comx-proto-fr.c +4 -1

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