[PATCH] pcmcia: handle __copy_from_user() return value in ioctl
authorDominik Brodowski <linux@dominikbrodowski.net>
Sun, 19 Nov 2006 16:21:27 +0000 (11:21 -0500)
committerDominik Brodowski <linux@dominikbrodowski.net>
Tue, 5 Dec 2006 01:09:16 +0000 (20:09 -0500)
Handle __copy_from_user() return value.

drivers/pcmcia/pcmcia_ioctl.c:597: warning: ignoring return value of '__copy_from_user', declared with attribute warn_unused_result

Noticed and first fix by Randy Dunlap.

Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
drivers/pcmcia/pcmcia_ioctl.c

index 310ede575caacd14dc38f122a4971d9221543bac..d077870c6731ae9ed63161829ee4bcb3fa2ba218 100644 (file)
@@ -594,7 +594,12 @@ static int ds_ioctl(struct inode * inode, struct file * file,
 
     err = ret = 0;
 
-    if (cmd & IOC_IN) __copy_from_user((char *)buf, uarg, size);
+    if (cmd & IOC_IN) {
+       if (__copy_from_user((char *)buf, uarg, size)) {
+           err = -EFAULT;
+           goto free_out;
+       }
+    }
 
     switch (cmd) {
     case DS_ADJUST_RESOURCE_INFO: