[PATCH 2/3] raw1394: __copy_from_user check

From: Daniel Drake
Date: Sun Nov 14 2004 - 16:16:32 EST


Add a check for the return value of __copy_to_user
Depends on the previous whitespace fix patch.

Signed-off-by: Daniel Drake <dsd@xxxxxxxxxx> --- linux/drivers/ieee1394/raw1394.c.orig 2004-11-14 03:02:30.000000000 +0000
+++ linux/drivers/ieee1394/raw1394.c 2004-11-14 03:12:12.928827600 +0000
@@ -447,9 +447,12 @@ static ssize_t raw1394_read(struct file
req->req.error = RAW1394_ERROR_MEMFAULT;
}
}
- __copy_to_user(buffer, &req->req, sizeof(req->req));

free_pending_request(req);
+
+ if (__copy_to_user(buffer, &req->req, sizeof(req->req)))
+ return -EFAULT;
+
return sizeof(struct raw1394_request);
}