Re: [PATCH] virtio_console: fix sparse warnings

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

 



On Mon, Dec 01, 2014 at 01:35:57PM +0200, Michael S. Tsirkin wrote:
> CHECK drivers/char/virtio_console.c
> drivers/char/virtio_console.c:687:36: warning: incorrect type in
> 	argument 1 (different address spaces)
> drivers/char/virtio_console.c:687:36:    expected void [noderef]
> 	<asn:1>*to
> drivers/char/virtio_console.c:687:36:    got char *out_buf
> drivers/char/virtio_console.c:790:35: warning: incorrect type in
> 	argument 2 (different address spaces)
> drivers/char/virtio_console.c:790:35:    expected char *out_buf
> drivers/char/virtio_console.c:790:35:    got char [noderef]
> 	<asn:1>*ubuf
> 
> fill_readbuf is reused with both kernel and userspace pointers,
> depending on value of to_user flag.

That's horrid, don't do that.  Have two different functions for this
type of thing, otherwise you defeat the whole purpose of trying to track
__user pointers.

thanks,

greg k-h
_______________________________________________
Virtualization mailing list
Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/virtualization




[Index of Archives]     [KVM Development]     [Libvirt Development]     [Libvirt Users]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux