RE: [PATCH -mm] updated megaraid gcc 4.1 warning fix

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,
Wednesday, May 10, 2006 1:27 PM, Daniel Walker wrote:
> drivers/scsi/megaraid.c: In function âEUR~megadev_ioctlâEUR(tm):
> drivers/scsi/megaraid.c:3665: warning: ignoring return value 
> of âEUR~copy_to_userâEUR(tm), declared with attribute warn_unused_result
Thank you for the patch. 
I accept the patch.

> -----Original Message-----
> From: Daniel Walker [mailto:[email protected]] 
> Sent: Wednesday, May 10, 2006 1:27 PM
> To: [email protected]
> Cc: Ju, Seokmann; [email protected]; 
> [email protected]
> Subject: [PATCH -mm] updated megaraid gcc 4.1 warning fix
> 
> Hows that Alan?
> 
> Fixes the following warning,
> 
> drivers/scsi/megaraid.c: In function âEUR~megadev_ioctlâEUR(tm):
> drivers/scsi/megaraid.c:3665: warning: ignoring return value 
> of âEUR~copy_to_userâEUR(tm), declared with attribute warn_unused_result
> 
> Signed-Off-By: Daniel Walker <[email protected]>
> 
> Index: linux-2.6.16/drivers/scsi/megaraid.c
> ===================================================================
> --- linux-2.6.16.orig/drivers/scsi/megaraid.c
> +++ linux-2.6.16/drivers/scsi/megaraid.c
> @@ -3662,8 +3662,9 @@ megadev_ioctl(struct inode *inode, struc
>  			 * Send the request sense data also, 
> irrespective of
>  			 * whether the user has asked for it or not.
>  			 */
> -			copy_to_user(upthru->reqsensearea,
> -					pthru->reqsensearea, 14);
> +			if (copy_to_user(upthru->reqsensearea,
> +					pthru->reqsensearea, 14))
> +				rval = (-EFAULT);
>  
>  freemem_and_return:
>  			if( pthru->dataxferlen ) {
> 
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux