Commit 89bddce5 authored by Stephen Hemminger's avatar Stephen Hemminger Committed by David S. Miller

[NET] socket: code style cleanup

Make socket.c conform to current style:
	* run through Lindent
	* get rid of unneeded casts
	* split assignment and comparsion where possible
Signed-off-by: default avatarStephen Hemminger <shemminger@osdl.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 53fad3cb
...@@ -99,13 +99,12 @@ static ssize_t sock_aio_read(struct kiocb *iocb, char __user *buf, ...@@ -99,13 +99,12 @@ static ssize_t sock_aio_read(struct kiocb *iocb, char __user *buf,
size_t size, loff_t pos); size_t size, loff_t pos);
static ssize_t sock_aio_write(struct kiocb *iocb, const char __user *buf, static ssize_t sock_aio_write(struct kiocb *iocb, const char __user *buf,
size_t size, loff_t pos); size_t size, loff_t pos);
static int sock_mmap(struct file *file, struct vm_area_struct * vma); static int sock_mmap(struct file *file, struct vm_area_struct *vma);
static int sock_close(struct inode *inode, struct file *file); static int sock_close(struct inode *inode, struct file *file);
static unsigned int sock_poll(struct file *file, static unsigned int sock_poll(struct file *file,
struct poll_table_struct *wait); struct poll_table_struct *wait);
static long sock_ioctl(struct file *file, static long sock_ioctl(struct file *file, unsigned int cmd, unsigned long arg);
unsigned int cmd, unsigned long arg);
#ifdef CONFIG_COMPAT #ifdef CONFIG_COMPAT
static long compat_sock_ioctl(struct file *file, static long compat_sock_ioctl(struct file *file,
unsigned int cmd, unsigned long arg); unsigned int cmd, unsigned long arg);
...@@ -193,7 +192,6 @@ static __inline__ void net_family_read_unlock(void) ...@@ -193,7 +192,6 @@ static __inline__ void net_family_read_unlock(void)
#define net_family_read_unlock() do { } while(0) #define net_family_read_unlock() do { } while(0)
#endif #endif
/* /*
* Statistics counters of the socket lists * Statistics counters of the socket lists
*/ */
...@@ -201,7 +199,8 @@ static __inline__ void net_family_read_unlock(void) ...@@ -201,7 +199,8 @@ static __inline__ void net_family_read_unlock(void)
static DEFINE_PER_CPU(int, sockets_in_use) = 0; static DEFINE_PER_CPU(int, sockets_in_use) = 0;
/* /*
* Support routines. Move socket addresses back and forth across the kernel/user * Support routines.
* Move socket addresses back and forth across the kernel/user
* divide and look after the messy bits. * divide and look after the messy bits.
*/ */
...@@ -227,11 +226,11 @@ static DEFINE_PER_CPU(int, sockets_in_use) = 0; ...@@ -227,11 +226,11 @@ static DEFINE_PER_CPU(int, sockets_in_use) = 0;
int move_addr_to_kernel(void __user *uaddr, int ulen, void *kaddr) int move_addr_to_kernel(void __user *uaddr, int ulen, void *kaddr)
{ {
if(ulen<0||ulen>MAX_SOCK_ADDR) if (ulen < 0 || ulen > MAX_SOCK_ADDR)
return -EINVAL; return -EINVAL;
if(ulen==0) if (ulen == 0)
return 0; return 0;
if(copy_from_user(kaddr,uaddr,ulen)) if (copy_from_user(kaddr, uaddr, ulen))
return -EFAULT; return -EFAULT;
return audit_sockaddr(ulen, kaddr); return audit_sockaddr(ulen, kaddr);
} }
...@@ -253,22 +252,23 @@ int move_addr_to_kernel(void __user *uaddr, int ulen, void *kaddr) ...@@ -253,22 +252,23 @@ int move_addr_to_kernel(void __user *uaddr, int ulen, void *kaddr)
* specified. Zero is returned for a success. * specified. Zero is returned for a success.
*/ */
int move_addr_to_user(void *kaddr, int klen, void __user *uaddr, int __user *ulen) int move_addr_to_user(void *kaddr, int klen, void __user *uaddr,
int __user *ulen)
{ {
int err; int err;
int len; int len;
if((err=get_user(len, ulen))) err = get_user(len, ulen);
if (err)
return err; return err;
if(len>klen) if (len > klen)
len=klen; len = klen;
if(len<0 || len> MAX_SOCK_ADDR) if (len < 0 || len > MAX_SOCK_ADDR)
return -EINVAL; return -EINVAL;
if(len) if (len) {
{
if (audit_sockaddr(klen, kaddr)) if (audit_sockaddr(klen, kaddr))
return -ENOMEM; return -ENOMEM;
if(copy_to_user(uaddr,kaddr,len)) if (copy_to_user(uaddr, kaddr, len))
return -EFAULT; return -EFAULT;
} }
/* /*
...@@ -280,12 +280,13 @@ int move_addr_to_user(void *kaddr, int klen, void __user *uaddr, int __user *ule ...@@ -280,12 +280,13 @@ int move_addr_to_user(void *kaddr, int klen, void __user *uaddr, int __user *ule
#define SOCKFS_MAGIC 0x534F434B #define SOCKFS_MAGIC 0x534F434B
static kmem_cache_t * sock_inode_cachep __read_mostly; static kmem_cache_t *sock_inode_cachep __read_mostly;
static struct inode *sock_alloc_inode(struct super_block *sb) static struct inode *sock_alloc_inode(struct super_block *sb)
{ {
struct socket_alloc *ei; struct socket_alloc *ei;
ei = (struct socket_alloc *)kmem_cache_alloc(sock_inode_cachep, SLAB_KERNEL);
ei = kmem_cache_alloc(sock_inode_cachep, SLAB_KERNEL);
if (!ei) if (!ei)
return NULL; return NULL;
init_waitqueue_head(&ei->socket.wait); init_waitqueue_head(&ei->socket.wait);
...@@ -307,12 +308,12 @@ static void sock_destroy_inode(struct inode *inode) ...@@ -307,12 +308,12 @@ static void sock_destroy_inode(struct inode *inode)
container_of(inode, struct socket_alloc, vfs_inode)); container_of(inode, struct socket_alloc, vfs_inode));
} }
static void init_once(void * foo, kmem_cache_t * cachep, unsigned long flags) static void init_once(void *foo, kmem_cache_t *cachep, unsigned long flags)
{ {
struct socket_alloc *ei = (struct socket_alloc *) foo; struct socket_alloc *ei = (struct socket_alloc *)foo;
if ((flags & (SLAB_CTOR_VERIFY|SLAB_CTOR_CONSTRUCTOR)) == if ((flags & (SLAB_CTOR_VERIFY|SLAB_CTOR_CONSTRUCTOR))
SLAB_CTOR_CONSTRUCTOR) == SLAB_CTOR_CONSTRUCTOR)
inode_init_once(&ei->vfs_inode); inode_init_once(&ei->vfs_inode);
} }
...@@ -320,9 +321,12 @@ static int init_inodecache(void) ...@@ -320,9 +321,12 @@ static int init_inodecache(void)
{ {
sock_inode_cachep = kmem_cache_create("sock_inode_cache", sock_inode_cachep = kmem_cache_create("sock_inode_cache",
sizeof(struct socket_alloc), sizeof(struct socket_alloc),
0, (SLAB_HWCACHE_ALIGN|SLAB_RECLAIM_ACCOUNT| 0,
(SLAB_HWCACHE_ALIGN |
SLAB_RECLAIM_ACCOUNT |
SLAB_MEM_SPREAD), SLAB_MEM_SPREAD),
init_once, NULL); init_once,
NULL);
if (sock_inode_cachep == NULL) if (sock_inode_cachep == NULL)
return -ENOMEM; return -ENOMEM;
return 0; return 0;
...@@ -335,7 +339,8 @@ static struct super_operations sockfs_ops = { ...@@ -335,7 +339,8 @@ static struct super_operations sockfs_ops = {
}; };
static int sockfs_get_sb(struct file_system_type *fs_type, static int sockfs_get_sb(struct file_system_type *fs_type,
int flags, const char *dev_name, void *data, struct vfsmount *mnt) int flags, const char *dev_name, void *data,
struct vfsmount *mnt)
{ {
return get_sb_pseudo(fs_type, "socket:", &sockfs_ops, SOCKFS_MAGIC, return get_sb_pseudo(fs_type, "socket:", &sockfs_ops, SOCKFS_MAGIC,
mnt); mnt);
...@@ -348,6 +353,7 @@ static struct file_system_type sock_fs_type = { ...@@ -348,6 +353,7 @@ static struct file_system_type sock_fs_type = {
.get_sb = sockfs_get_sb, .get_sb = sockfs_get_sb,
.kill_sb = kill_anon_super, .kill_sb = kill_anon_super,
}; };
static int sockfs_delete_dentry(struct dentry *dentry) static int sockfs_delete_dentry(struct dentry *dentry)
{ {
return 1; return 1;
...@@ -477,10 +483,12 @@ struct socket *sockfd_lookup(int fd, int *err) ...@@ -477,10 +483,12 @@ struct socket *sockfd_lookup(int fd, int *err)
struct file *file; struct file *file;
struct socket *sock; struct socket *sock;
if (!(file = fget(fd))) { file = fget(fd);
if (!file) {
*err = -EBADF; *err = -EBADF;
return NULL; return NULL;
} }
sock = sock_from_file(file, err); sock = sock_from_file(file, err);
if (!sock) if (!sock)
fput(file); fput(file);
...@@ -513,8 +521,8 @@ static struct socket *sockfd_lookup_light(int fd, int *err, int *fput_needed) ...@@ -513,8 +521,8 @@ static struct socket *sockfd_lookup_light(int fd, int *err, int *fput_needed)
static struct socket *sock_alloc(void) static struct socket *sock_alloc(void)
{ {
struct inode * inode; struct inode *inode;
struct socket * sock; struct socket *sock;
inode = new_inode(sock_mnt->mnt_sb); inode = new_inode(sock_mnt->mnt_sb);
if (!inode) if (!inode)
...@@ -522,7 +530,7 @@ static struct socket *sock_alloc(void) ...@@ -522,7 +530,7 @@ static struct socket *sock_alloc(void)
sock = SOCKET_I(inode); sock = SOCKET_I(inode);
inode->i_mode = S_IFSOCK|S_IRWXUGO; inode->i_mode = S_IFSOCK | S_IRWXUGO;
inode->i_uid = current->fsuid; inode->i_uid = current->fsuid;
inode->i_gid = current->fsgid; inode->i_gid = current->fsgid;
...@@ -575,7 +583,7 @@ void sock_release(struct socket *sock) ...@@ -575,7 +583,7 @@ void sock_release(struct socket *sock)
iput(SOCK_INODE(sock)); iput(SOCK_INODE(sock));
return; return;
} }
sock->file=NULL; sock->file = NULL;
} }
static inline int __sock_sendmsg(struct kiocb *iocb, struct socket *sock, static inline int __sock_sendmsg(struct kiocb *iocb, struct socket *sock,
...@@ -621,7 +629,7 @@ int kernel_sendmsg(struct socket *sock, struct msghdr *msg, ...@@ -621,7 +629,7 @@ int kernel_sendmsg(struct socket *sock, struct msghdr *msg,
* the following is safe, since for compiler definitions of kvec and * the following is safe, since for compiler definitions of kvec and
* iovec are identical, yielding the same in-core layout and alignment * iovec are identical, yielding the same in-core layout and alignment
*/ */
msg->msg_iov = (struct iovec *)vec, msg->msg_iov = (struct iovec *)vec;
msg->msg_iovlen = num; msg->msg_iovlen = num;
result = sock_sendmsg(sock, msg, size); result = sock_sendmsg(sock, msg, size);
set_fs(oldfs); set_fs(oldfs);
...@@ -663,8 +671,7 @@ int sock_recvmsg(struct socket *sock, struct msghdr *msg, ...@@ -663,8 +671,7 @@ int sock_recvmsg(struct socket *sock, struct msghdr *msg,
} }
int kernel_recvmsg(struct socket *sock, struct msghdr *msg, int kernel_recvmsg(struct socket *sock, struct msghdr *msg,
struct kvec *vec, size_t num, struct kvec *vec, size_t num, size_t size, int flags)
size_t size, int flags)
{ {
mm_segment_t oldfs = get_fs(); mm_segment_t oldfs = get_fs();
int result; int result;
...@@ -674,8 +681,7 @@ int kernel_recvmsg(struct socket *sock, struct msghdr *msg, ...@@ -674,8 +681,7 @@ int kernel_recvmsg(struct socket *sock, struct msghdr *msg,
* the following is safe, since for compiler definitions of kvec and * the following is safe, since for compiler definitions of kvec and
* iovec are identical, yielding the same in-core layout and alignment * iovec are identical, yielding the same in-core layout and alignment
*/ */
msg->msg_iov = (struct iovec *)vec, msg->msg_iov = (struct iovec *)vec, msg->msg_iovlen = num;
msg->msg_iovlen = num;
result = sock_recvmsg(sock, msg, size, flags); result = sock_recvmsg(sock, msg, size, flags);
set_fs(oldfs); set_fs(oldfs);
return result; return result;
...@@ -702,7 +708,8 @@ static ssize_t sock_sendpage(struct file *file, struct page *page, ...@@ -702,7 +708,8 @@ static ssize_t sock_sendpage(struct file *file, struct page *page,
} }
static struct sock_iocb *alloc_sock_iocb(struct kiocb *iocb, static struct sock_iocb *alloc_sock_iocb(struct kiocb *iocb,
char __user *ubuf, size_t size, struct sock_iocb *siocb) char __user *ubuf, size_t size,
struct sock_iocb *siocb)
{ {
if (!is_sync_kiocb(iocb)) { if (!is_sync_kiocb(iocb)) {
siocb = kmalloc(sizeof(*siocb), GFP_KERNEL); siocb = kmalloc(sizeof(*siocb), GFP_KERNEL);
...@@ -720,20 +727,21 @@ static struct sock_iocb *alloc_sock_iocb(struct kiocb *iocb, ...@@ -720,20 +727,21 @@ static struct sock_iocb *alloc_sock_iocb(struct kiocb *iocb,
} }
static ssize_t do_sock_read(struct msghdr *msg, struct kiocb *iocb, static ssize_t do_sock_read(struct msghdr *msg, struct kiocb *iocb,
struct file *file, struct iovec *iov, unsigned long nr_segs) struct file *file, struct iovec *iov,
unsigned long nr_segs)
{ {
struct socket *sock = file->private_data; struct socket *sock = file->private_data;
size_t size = 0; size_t size = 0;
int i; int i;
for (i = 0 ; i < nr_segs ; i++) for (i = 0; i < nr_segs; i++)
size += iov[i].iov_len; size += iov[i].iov_len;
msg->msg_name = NULL; msg->msg_name = NULL;
msg->msg_namelen = 0; msg->msg_namelen = 0;
msg->msg_control = NULL; msg->msg_control = NULL;
msg->msg_controllen = 0; msg->msg_controllen = 0;
msg->msg_iov = (struct iovec *) iov; msg->msg_iov = (struct iovec *)iov;
msg->msg_iovlen = nr_segs; msg->msg_iovlen = nr_segs;
msg->msg_flags = (file->f_flags & O_NONBLOCK) ? MSG_DONTWAIT : 0; msg->msg_flags = (file->f_flags & O_NONBLOCK) ? MSG_DONTWAIT : 0;
...@@ -775,20 +783,21 @@ static ssize_t sock_aio_read(struct kiocb *iocb, char __user *ubuf, ...@@ -775,20 +783,21 @@ static ssize_t sock_aio_read(struct kiocb *iocb, char __user *ubuf,
} }
static ssize_t do_sock_write(struct msghdr *msg, struct kiocb *iocb, static ssize_t do_sock_write(struct msghdr *msg, struct kiocb *iocb,
struct file *file, struct iovec *iov, unsigned long nr_segs) struct file *file, struct iovec *iov,
unsigned long nr_segs)
{ {
struct socket *sock = file->private_data; struct socket *sock = file->private_data;
size_t size = 0; size_t size = 0;
int i; int i;
for (i = 0 ; i < nr_segs ; i++) for (i = 0; i < nr_segs; i++)
size += iov[i].iov_len; size += iov[i].iov_len;
msg->msg_name = NULL; msg->msg_name = NULL;
msg->msg_namelen = 0; msg->msg_namelen = 0;
msg->msg_control = NULL; msg->msg_control = NULL;
msg->msg_controllen = 0; msg->msg_controllen = 0;
msg->msg_iov = (struct iovec *) iov; msg->msg_iov = (struct iovec *)iov;
msg->msg_iovlen = nr_segs; msg->msg_iovlen = nr_segs;
msg->msg_flags = (file->f_flags & O_NONBLOCK) ? MSG_DONTWAIT : 0; msg->msg_flags = (file->f_flags & O_NONBLOCK) ? MSG_DONTWAIT : 0;
if (sock->type == SOCK_SEQPACKET) if (sock->type == SOCK_SEQPACKET)
...@@ -832,43 +841,45 @@ static ssize_t sock_aio_write(struct kiocb *iocb, const char __user *ubuf, ...@@ -832,43 +841,45 @@ static ssize_t sock_aio_write(struct kiocb *iocb, const char __user *ubuf,
&x->async_iov, 1); &x->async_iov, 1);
} }
/* /*
* Atomic setting of ioctl hooks to avoid race * Atomic setting of ioctl hooks to avoid race
* with module unload. * with module unload.
*/ */
static DEFINE_MUTEX(br_ioctl_mutex); static DEFINE_MUTEX(br_ioctl_mutex);
static int (*br_ioctl_hook)(unsigned int cmd, void __user *arg) = NULL; static int (*br_ioctl_hook) (unsigned int cmd, void __user *arg) = NULL;
void brioctl_set(int (*hook)(unsigned int, void __user *)) void brioctl_set(int (*hook) (unsigned int, void __user *))
{ {
mutex_lock(&br_ioctl_mutex); mutex_lock(&br_ioctl_mutex);
br_ioctl_hook = hook; br_ioctl_hook = hook;
mutex_unlock(&br_ioctl_mutex); mutex_unlock(&br_ioctl_mutex);
} }
EXPORT_SYMBOL(brioctl_set); EXPORT_SYMBOL(brioctl_set);
static DEFINE_MUTEX(vlan_ioctl_mutex); static DEFINE_MUTEX(vlan_ioctl_mutex);
static int (*vlan_ioctl_hook)(void __user *arg); static int (*vlan_ioctl_hook) (void __user *arg);
void vlan_ioctl_set(int (*hook)(void __user *)) void vlan_ioctl_set(int (*hook) (void __user *))
{ {
mutex_lock(&vlan_ioctl_mutex); mutex_lock(&vlan_ioctl_mutex);
vlan_ioctl_hook = hook; vlan_ioctl_hook = hook;
mutex_unlock(&vlan_ioctl_mutex); mutex_unlock(&vlan_ioctl_mutex);
} }
EXPORT_SYMBOL(vlan_ioctl_set); EXPORT_SYMBOL(vlan_ioctl_set);
static DEFINE_MUTEX(dlci_ioctl_mutex); static DEFINE_MUTEX(dlci_ioctl_mutex);
static int (*dlci_ioctl_hook)(unsigned int, void __user *); static int (*dlci_ioctl_hook) (unsigned int, void __user *);
void dlci_ioctl_set(int (*hook)(unsigned int, void __user *)) void dlci_ioctl_set(int (*hook) (unsigned int, void __user *))
{ {
mutex_lock(&dlci_ioctl_mutex); mutex_lock(&dlci_ioctl_mutex);
dlci_ioctl_hook = hook; dlci_ioctl_hook = hook;
mutex_unlock(&dlci_ioctl_mutex); mutex_unlock(&dlci_ioctl_mutex);
} }
EXPORT_SYMBOL(dlci_ioctl_set); EXPORT_SYMBOL(dlci_ioctl_set);
/* /*
...@@ -901,7 +912,8 @@ static long sock_ioctl(struct file *file, unsigned cmd, unsigned long arg) ...@@ -901,7 +912,8 @@ static long sock_ioctl(struct file *file, unsigned cmd, unsigned long arg)
break; break;
case FIOGETOWN: case FIOGETOWN:
case SIOCGPGRP: case SIOCGPGRP:
err = put_user(sock->file->f_owner.pid, (int __user *)argp); err = put_user(sock->file->f_owner.pid,
(int __user *)argp);
break; break;
case SIOCGIFBR: case SIOCGIFBR:
case SIOCSIFBR: case SIOCSIFBR:
...@@ -988,7 +1000,7 @@ out_release: ...@@ -988,7 +1000,7 @@ out_release:
} }
/* No kernel lock held - perfect */ /* No kernel lock held - perfect */
static unsigned int sock_poll(struct file *file, poll_table * wait) static unsigned int sock_poll(struct file *file, poll_table *wait)
{ {
struct socket *sock; struct socket *sock;
...@@ -999,7 +1011,7 @@ static unsigned int sock_poll(struct file *file, poll_table * wait) ...@@ -999,7 +1011,7 @@ static unsigned int sock_poll(struct file *file, poll_table * wait)
return sock->ops->poll(file, sock, wait); return sock->ops->poll(file, sock, wait);
} }
static int sock_mmap(struct file * file, struct vm_area_struct * vma) static int sock_mmap(struct file *file, struct vm_area_struct *vma)
{ {
struct socket *sock = file->private_data; struct socket *sock = file->private_data;
...@@ -1013,8 +1025,7 @@ static int sock_close(struct inode *inode, struct file *filp) ...@@ -1013,8 +1025,7 @@ static int sock_close(struct inode *inode, struct file *filp)
* closing an unfinished socket. * closing an unfinished socket.
*/ */
if (!inode) if (!inode) {
{
printk(KERN_DEBUG "sock_close: NULL inode\n"); printk(KERN_DEBUG "sock_close: NULL inode\n");
return 0; return 0;
} }
...@@ -1040,57 +1051,52 @@ static int sock_close(struct inode *inode, struct file *filp) ...@@ -1040,57 +1051,52 @@ static int sock_close(struct inode *inode, struct file *filp)
static int sock_fasync(int fd, struct file *filp, int on) static int sock_fasync(int fd, struct file *filp, int on)
{ {
struct fasync_struct *fa, *fna=NULL, **prev; struct fasync_struct *fa, *fna = NULL, **prev;
struct socket *sock; struct socket *sock;
struct sock *sk; struct sock *sk;
if (on) if (on) {
{
fna = kmalloc(sizeof(struct fasync_struct), GFP_KERNEL); fna = kmalloc(sizeof(struct fasync_struct), GFP_KERNEL);
if(fna==NULL) if (fna == NULL)
return -ENOMEM; return -ENOMEM;
} }
sock = filp->private_data; sock = filp->private_data;
if ((sk=sock->sk) == NULL) { sk = sock->sk;
if (sk == NULL) {
kfree(fna); kfree(fna);
return -EINVAL; return -EINVAL;
} }
lock_sock(sk); lock_sock(sk);
prev=&(sock->fasync_list); prev = &(sock->fasync_list);
for (fa=*prev; fa!=NULL; prev=&fa->fa_next,fa=*prev) for (fa = *prev; fa != NULL; prev = &fa->fa_next, fa = *prev)
if (fa->fa_file==filp) if (fa->fa_file == filp)
break; break;
if(on) if (on) {
{ if (fa != NULL) {
if(fa!=NULL)
{
write_lock_bh(&sk->sk_callback_lock); write_lock_bh(&sk->sk_callback_lock);
fa->fa_fd=fd; fa->fa_fd = fd;
write_unlock_bh(&sk->sk_callback_lock); write_unlock_bh(&sk->sk_callback_lock);
kfree(fna); kfree(fna);
goto out; goto out;
} }
fna->fa_file=filp; fna->fa_file = filp;
fna->fa_fd=fd; fna->fa_fd = fd;
fna->magic=FASYNC_MAGIC; fna->magic = FASYNC_MAGIC;
fna->fa_next=sock->fasync_list; fna->fa_next = sock->fasync_list;
write_lock_bh(&sk->sk_callback_lock); write_lock_bh(&sk->sk_callback_lock);
sock->fasync_list=fna; sock->fasync_list = fna;
write_unlock_bh(&sk->sk_callback_lock); write_unlock_bh(&sk->sk_callback_lock);
} } else {
else if (fa != NULL) {
{
if (fa!=NULL)
{
write_lock_bh(&sk->sk_callback_lock); write_lock_bh(&sk->sk_callback_lock);
*prev=fa->fa_next; *prev = fa->fa_next;
write_unlock_bh(&sk->sk_callback_lock); write_unlock_bh(&sk->sk_callback_lock);
kfree(fa); kfree(fa);
} }
...@@ -1107,8 +1113,7 @@ int sock_wake_async(struct socket *sock, int how, int band) ...@@ -1107,8 +1113,7 @@ int sock_wake_async(struct socket *sock, int how, int band)
{ {
if (!sock || !sock->fasync_list) if (!sock || !sock->fasync_list)
return -1; return -1;
switch (how) switch (how) {
{
case 1: case 1:
if (test_bit(SOCK_ASYNC_WAITDATA, &sock->flags)) if (test_bit(SOCK_ASYNC_WAITDATA, &sock->flags))
...@@ -1119,7 +1124,7 @@ int sock_wake_async(struct socket *sock, int how, int band) ...@@ -1119,7 +1124,7 @@ int sock_wake_async(struct socket *sock, int how, int band)
break; break;
/* fall through */ /* fall through */
case 0: case 0:
call_kill: call_kill:
__kill_fasync(sock->fasync_list, SIGIO, band); __kill_fasync(sock->fasync_list, SIGIO, band);
break; break;
case 3: case 3:
...@@ -1128,7 +1133,8 @@ int sock_wake_async(struct socket *sock, int how, int band) ...@@ -1128,7 +1133,8 @@ int sock_wake_async(struct socket *sock, int how, int band)
return 0; return 0;
} }
static int __sock_create(int family, int type, int protocol, struct socket **res, int kern) static int __sock_create(int family, int type, int protocol,
struct socket **res, int kern)
{ {
int err; int err;
struct socket *sock; struct socket *sock;
...@@ -1150,7 +1156,8 @@ static int __sock_create(int family, int type, int protocol, struct socket **res ...@@ -1150,7 +1156,8 @@ static int __sock_create(int family, int type, int protocol, struct socket **res
static int warned; static int warned;
if (!warned) { if (!warned) {
warned = 1; warned = 1;
printk(KERN_INFO "%s uses obsolete (PF_INET,SOCK_PACKET)\n", current->comm); printk(KERN_INFO "%s uses obsolete (PF_INET,SOCK_PACKET)\n",
current->comm);
} }
family = PF_PACKET; family = PF_PACKET;
} }
...@@ -1166,9 +1173,8 @@ static int __sock_create(int family, int type, int protocol, struct socket **res ...@@ -1166,9 +1173,8 @@ static int __sock_create(int family, int type, int protocol, struct socket **res
* requested real, full-featured networking support upon configuration. * requested real, full-featured networking support upon configuration.
* Otherwise module support will break! * Otherwise module support will break!
*/ */
if (net_families[family]==NULL) if (net_families[family] == NULL) {
{ request_module("net-pf-%d", family);
request_module("net-pf-%d",family);
} }
#endif #endif
...@@ -1271,7 +1277,8 @@ out_release: ...@@ -1271,7 +1277,8 @@ out_release:
* Create a pair of connected sockets. * Create a pair of connected sockets.
*/ */
asmlinkage long sys_socketpair(int family, int type, int protocol, int __user *usockvec) asmlinkage long sys_socketpair(int family, int type, int protocol,
int __user *usockvec)
{ {
struct socket *sock1, *sock2; struct socket *sock1, *sock2;
int fd1, fd2, err; int fd1, fd2, err;
...@@ -1332,7 +1339,6 @@ out: ...@@ -1332,7 +1339,6 @@ out:
return err; return err;
} }
/* /*
* Bind a name to a socket. Nothing much to do here since it's * Bind a name to a socket. Nothing much to do here since it's
* the protocol's responsibility to handle the local address. * the protocol's responsibility to handle the local address.
...@@ -1347,20 +1353,23 @@ asmlinkage long sys_bind(int fd, struct sockaddr __user *umyaddr, int addrlen) ...@@ -1347,20 +1353,23 @@ asmlinkage long sys_bind(int fd, struct sockaddr __user *umyaddr, int addrlen)
char address[MAX_SOCK_ADDR]; char address[MAX_SOCK_ADDR];
int err, fput_needed; int err, fput_needed;
if((sock = sockfd_lookup_light(fd, &err, &fput_needed))!=NULL) sock = sockfd_lookup_light(fd, &err, &fput_needed);
{ if(sock) {
if((err=move_addr_to_kernel(umyaddr,addrlen,address))>=0) { err = move_addr_to_kernel(umyaddr, addrlen, address);
err = security_socket_bind(sock, (struct sockaddr *)address, addrlen); if (err >= 0) {
err = security_socket_bind(sock,
(struct sockaddr *)address,
addrlen);
if (!err) if (!err)
err = sock->ops->bind(sock, err = sock->ops->bind(sock,
(struct sockaddr *)address, addrlen); (struct sockaddr *)
address, addrlen);
} }
fput_light(sock->file, fput_needed); fput_light(sock->file, fput_needed);
} }
return err; return err;
} }
/* /*
* Perform a listen. Basically, we allow the protocol to do anything * Perform a listen. Basically, we allow the protocol to do anything
* necessary for a listen, and if that works, we mark the socket as * necessary for a listen, and if that works, we mark the socket as
...@@ -1374,8 +1383,9 @@ asmlinkage long sys_listen(int fd, int backlog) ...@@ -1374,8 +1383,9 @@ asmlinkage long sys_listen(int fd, int backlog)
struct socket *sock; struct socket *sock;
int err, fput_needed; int err, fput_needed;
if ((sock = sockfd_lookup_light(fd, &err, &fput_needed)) != NULL) { sock = sockfd_lookup_light(fd, &err, &fput_needed);
if ((unsigned) backlog > sysctl_somaxconn) if (sock) {
if ((unsigned)backlog > sysctl_somaxconn)
backlog = sysctl_somaxconn; backlog = sysctl_somaxconn;
err = security_socket_listen(sock, backlog); err = security_socket_listen(sock, backlog);
...@@ -1387,7 +1397,6 @@ asmlinkage long sys_listen(int fd, int backlog) ...@@ -1387,7 +1397,6 @@ asmlinkage long sys_listen(int fd, int backlog)
return err; return err;
} }
/* /*
* For accept, we attempt to create a new socket, set up the link * For accept, we attempt to create a new socket, set up the link
* with the client, wake up the client, then return the new * with the client, wake up the client, then return the new
...@@ -1400,7 +1409,8 @@ asmlinkage long sys_listen(int fd, int backlog) ...@@ -1400,7 +1409,8 @@ asmlinkage long sys_listen(int fd, int backlog)
* clean when we restucture accept also. * clean when we restucture accept also.
*/ */
asmlinkage long sys_accept(int fd, struct sockaddr __user *upeer_sockaddr, int __user *upeer_addrlen) asmlinkage long sys_accept(int fd, struct sockaddr __user *upeer_sockaddr,
int __user *upeer_addrlen)
{ {
struct socket *sock, *newsock; struct socket *sock, *newsock;
struct file *newfile; struct file *newfile;
...@@ -1444,11 +1454,13 @@ asmlinkage long sys_accept(int fd, struct sockaddr __user *upeer_sockaddr, int _ ...@@ -1444,11 +1454,13 @@ asmlinkage long sys_accept(int fd, struct sockaddr __user *upeer_sockaddr, int _
goto out_fd; goto out_fd;
if (upeer_sockaddr) { if (upeer_sockaddr) {
if(newsock->ops->getname(newsock, (struct sockaddr *)address, &len, 2)<0) { if (newsock->ops->getname(newsock, (struct sockaddr *)address,
&len, 2) < 0) {
err = -ECONNABORTED; err = -ECONNABORTED;
goto out_fd; goto out_fd;
} }
err = move_addr_to_user(address, len, upeer_sockaddr, upeer_addrlen); err = move_addr_to_user(address, len, upeer_sockaddr,
upeer_addrlen);
if (err < 0) if (err < 0)
goto out_fd; goto out_fd;
} }
...@@ -1470,7 +1482,6 @@ out_fd: ...@@ -1470,7 +1482,6 @@ out_fd:
goto out_put; goto out_put;
} }
/* /*
* Attempt to connect to a socket with the server address. The address * Attempt to connect to a socket with the server address. The address
* is in user space so we verify it is OK and move it to kernel space. * is in user space so we verify it is OK and move it to kernel space.
...@@ -1483,7 +1494,8 @@ out_fd: ...@@ -1483,7 +1494,8 @@ out_fd:
* include the -EINPROGRESS status for such sockets. * include the -EINPROGRESS status for such sockets.
*/ */
asmlinkage long sys_connect(int fd, struct sockaddr __user *uservaddr, int addrlen) asmlinkage long sys_connect(int fd, struct sockaddr __user *uservaddr,
int addrlen)
{ {
struct socket *sock; struct socket *sock;
char address[MAX_SOCK_ADDR]; char address[MAX_SOCK_ADDR];
...@@ -1496,11 +1508,12 @@ asmlinkage long sys_connect(int fd, struct sockaddr __user *uservaddr, int addrl ...@@ -1496,11 +1508,12 @@ asmlinkage long sys_connect(int fd, struct sockaddr __user *uservaddr, int addrl
if (err < 0) if (err < 0)
goto out_put; goto out_put;
err = security_socket_connect(sock, (struct sockaddr *)address, addrlen); err =
security_socket_connect(sock, (struct sockaddr *)address, addrlen);
if (err) if (err)
goto out_put; goto out_put;
err = sock->ops->connect(sock, (struct sockaddr *) address, addrlen, err = sock->ops->connect(sock, (struct sockaddr *)address, addrlen,
sock->file->f_flags); sock->file->f_flags);
out_put: out_put:
fput_light(sock->file, fput_needed); fput_light(sock->file, fput_needed);
...@@ -1513,7 +1526,8 @@ out: ...@@ -1513,7 +1526,8 @@ out:
* name to user space. * name to user space.
*/ */
asmlinkage long sys_getsockname(int fd, struct sockaddr __user *usockaddr, int __user *usockaddr_len) asmlinkage long sys_getsockname(int fd, struct sockaddr __user *usockaddr,
int __user *usockaddr_len)
{ {
struct socket *sock; struct socket *sock;
char address[MAX_SOCK_ADDR]; char address[MAX_SOCK_ADDR];
...@@ -1543,22 +1557,27 @@ out: ...@@ -1543,22 +1557,27 @@ out:
* name to user space. * name to user space.
*/ */
asmlinkage long sys_getpeername(int fd, struct sockaddr __user *usockaddr, int __user *usockaddr_len) asmlinkage long sys_getpeername(int fd, struct sockaddr __user *usockaddr,
int __user *usockaddr_len)
{ {
struct socket *sock; struct socket *sock;
char address[MAX_SOCK_ADDR]; char address[MAX_SOCK_ADDR];
int len, err, fput_needed; int len, err, fput_needed;
if ((sock = sockfd_lookup_light(fd, &err, &fput_needed)) != NULL) { sock = sockfd_lookup_light(fd, &err, &fput_needed);
if (sock != NULL) {
err = security_socket_getpeername(sock); err = security_socket_getpeername(sock);
if (err) { if (err) {
fput_light(sock->file, fput_needed); fput_light(sock->file, fput_needed);
return err; return err;
} }
err = sock->ops->getname(sock, (struct sockaddr *)address, &len, 1); err =
sock->ops->getname(sock, (struct sockaddr *)address, &len,
1);
if (!err) if (!err)
err=move_addr_to_user(address,len, usockaddr, usockaddr_len); err = move_addr_to_user(address, len, usockaddr,
usockaddr_len);
fput_light(sock->file, fput_needed); fput_light(sock->file, fput_needed);
} }
return err; return err;
...@@ -1570,8 +1589,9 @@ asmlinkage long sys_getpeername(int fd, struct sockaddr __user *usockaddr, int _ ...@@ -1570,8 +1589,9 @@ asmlinkage long sys_getpeername(int fd, struct sockaddr __user *usockaddr, int _
* the protocol. * the protocol.
*/ */
asmlinkage long sys_sendto(int fd, void __user * buff, size_t len, unsigned flags, asmlinkage long sys_sendto(int fd, void __user *buff, size_t len,
struct sockaddr __user *addr, int addr_len) unsigned flags, struct sockaddr __user *addr,
int addr_len)
{ {
struct socket *sock; struct socket *sock;
char address[MAX_SOCK_ADDR]; char address[MAX_SOCK_ADDR];
...@@ -1588,20 +1608,20 @@ asmlinkage long sys_sendto(int fd, void __user * buff, size_t len, unsigned flag ...@@ -1588,20 +1608,20 @@ asmlinkage long sys_sendto(int fd, void __user * buff, size_t len, unsigned flag
sock = sock_from_file(sock_file, &err); sock = sock_from_file(sock_file, &err);
if (!sock) if (!sock)
goto out_put; goto out_put;
iov.iov_base=buff; iov.iov_base = buff;
iov.iov_len=len; iov.iov_len = len;
msg.msg_name=NULL; msg.msg_name = NULL;
msg.msg_iov=&iov; msg.msg_iov = &iov;
msg.msg_iovlen=1; msg.msg_iovlen = 1;
msg.msg_control=NULL; msg.msg_control = NULL;
msg.msg_controllen=0; msg.msg_controllen = 0;
msg.msg_namelen=0; msg.msg_namelen = 0;
if (addr) { if (addr) {
err = move_addr_to_kernel(addr, addr_len, address); err = move_addr_to_kernel(addr, addr_len, address);
if (err < 0) if (err < 0)
goto out_put; goto out_put;
msg.msg_name=address; msg.msg_name = address;
msg.msg_namelen=addr_len; msg.msg_namelen = addr_len;
} }
if (sock->file->f_flags & O_NONBLOCK) if (sock->file->f_flags & O_NONBLOCK)
flags |= MSG_DONTWAIT; flags |= MSG_DONTWAIT;
...@@ -1617,7 +1637,7 @@ out_put: ...@@ -1617,7 +1637,7 @@ out_put:
* Send a datagram down a socket. * Send a datagram down a socket.
*/ */
asmlinkage long sys_send(int fd, void __user * buff, size_t len, unsigned flags) asmlinkage long sys_send(int fd, void __user *buff, size_t len, unsigned flags)
{ {
return sys_sendto(fd, buff, len, flags, NULL, 0); return sys_sendto(fd, buff, len, flags, NULL, 0);
} }
...@@ -1628,14 +1648,15 @@ asmlinkage long sys_send(int fd, void __user * buff, size_t len, unsigned flags) ...@@ -1628,14 +1648,15 @@ asmlinkage long sys_send(int fd, void __user * buff, size_t len, unsigned flags)
* sender address from kernel to user space. * sender address from kernel to user space.
*/ */
asmlinkage long sys_recvfrom(int fd, void __user * ubuf, size_t size, unsigned flags, asmlinkage long sys_recvfrom(int fd, void __user *ubuf, size_t size,
struct sockaddr __user *addr, int __user *addr_len) unsigned flags, struct sockaddr __user *addr,
int __user *addr_len)
{ {
struct socket *sock; struct socket *sock;
struct iovec iov; struct iovec iov;
struct msghdr msg; struct msghdr msg;
char address[MAX_SOCK_ADDR]; char address[MAX_SOCK_ADDR];
int err,err2; int err, err2;
struct file *sock_file; struct file *sock_file;
int fput_needed; int fput_needed;
...@@ -1647,23 +1668,22 @@ asmlinkage long sys_recvfrom(int fd, void __user * ubuf, size_t size, unsigned f ...@@ -1647,23 +1668,22 @@ asmlinkage long sys_recvfrom(int fd, void __user * ubuf, size_t size, unsigned f
if (!sock) if (!sock)
goto out; goto out;
msg.msg_control=NULL; msg.msg_control = NULL;
msg.msg_controllen=0; msg.msg_controllen = 0;
msg.msg_iovlen=1; msg.msg_iovlen = 1;
msg.msg_iov=&iov; msg.msg_iov = &iov;
iov.iov_len=size; iov.iov_len = size;
iov.iov_base=ubuf; iov.iov_base = ubuf;
msg.msg_name=address; msg.msg_name = address;
msg.msg_namelen=MAX_SOCK_ADDR; msg.msg_namelen = MAX_SOCK_ADDR;
if (sock->file->f_flags & O_NONBLOCK) if (sock->file->f_flags & O_NONBLOCK)
flags |= MSG_DONTWAIT; flags |= MSG_DONTWAIT;
err=sock_recvmsg(sock, &msg, size, flags); err = sock_recvmsg(sock, &msg, size, flags);
if(err >= 0 && addr != NULL) if (err >= 0 && addr != NULL) {
{ err2 = move_addr_to_user(address, msg.msg_namelen, addr, addr_len);
err2=move_addr_to_user(address, msg.msg_namelen, addr, addr_len); if (err2 < 0)
if(err2<0) err = err2;
err=err2;
} }
out: out:
fput_light(sock_file, fput_needed); fput_light(sock_file, fput_needed);
...@@ -1674,7 +1694,8 @@ out: ...@@ -1674,7 +1694,8 @@ out:
* Receive a datagram from a socket. * Receive a datagram from a socket.
*/ */
asmlinkage long sys_recv(int fd, void __user * ubuf, size_t size, unsigned flags) asmlinkage long sys_recv(int fd, void __user *ubuf, size_t size,
unsigned flags)
{ {
return sys_recvfrom(fd, ubuf, size, flags, NULL, NULL); return sys_recvfrom(fd, ubuf, size, flags, NULL, NULL);
} }
...@@ -1684,7 +1705,8 @@ asmlinkage long sys_recv(int fd, void __user * ubuf, size_t size, unsigned flags ...@@ -1684,7 +1705,8 @@ asmlinkage long sys_recv(int fd, void __user * ubuf, size_t size, unsigned flags
* to pass the user mode parameter for the protocols to sort out. * to pass the user mode parameter for the protocols to sort out.
*/ */
asmlinkage long sys_setsockopt(int fd, int level, int optname, char __user *optval, int optlen) asmlinkage long sys_setsockopt(int fd, int level, int optname,
char __user *optval, int optlen)
{ {
int err, fput_needed; int err, fput_needed;
struct socket *sock; struct socket *sock;
...@@ -1692,16 +1714,20 @@ asmlinkage long sys_setsockopt(int fd, int level, int optname, char __user *optv ...@@ -1692,16 +1714,20 @@ asmlinkage long sys_setsockopt(int fd, int level, int optname, char __user *optv
if (optlen < 0) if (optlen < 0)
return -EINVAL; return -EINVAL;
if ((sock = sockfd_lookup_light(fd, &err, &fput_needed)) != NULL) sock = sockfd_lookup_light(fd, &err, &fput_needed);
{ if (sock != NULL) {
err = security_socket_setsockopt(sock,level,optname); err = security_socket_setsockopt(sock, level, optname);
if (err) if (err)
goto out_put; goto out_put;
if (level == SOL_SOCKET) if (level == SOL_SOCKET)
err=sock_setsockopt(sock,level,optname,optval,optlen); err =
sock_setsockopt(sock, level, optname, optval,
optlen);
else else
err=sock->ops->setsockopt(sock, level, optname, optval, optlen); err =
sock->ops->setsockopt(sock, level, optname, optval,
optlen);
out_put: out_put:
fput_light(sock->file, fput_needed); fput_light(sock->file, fput_needed);
} }
...@@ -1713,27 +1739,32 @@ out_put: ...@@ -1713,27 +1739,32 @@ out_put:
* to pass a user mode parameter for the protocols to sort out. * to pass a user mode parameter for the protocols to sort out.
*/ */
asmlinkage long sys_getsockopt(int fd, int level, int optname, char __user *optval, int __user *optlen) asmlinkage long sys_getsockopt(int fd, int level, int optname,
char __user *optval, int __user *optlen)
{ {
int err, fput_needed; int err, fput_needed;
struct socket *sock; struct socket *sock;
if ((sock = sockfd_lookup_light(fd, &err, &fput_needed)) != NULL) { sock = sockfd_lookup_light(fd, &err, &fput_needed);
if (sock != NULL) {
err = security_socket_getsockopt(sock, level, optname); err = security_socket_getsockopt(sock, level, optname);
if (err) if (err)
goto out_put; goto out_put;
if (level == SOL_SOCKET) if (level == SOL_SOCKET)
err=sock_getsockopt(sock,level,optname,optval,optlen); err =
sock_getsockopt(sock, level, optname, optval,
optlen);
else else
err=sock->ops->getsockopt(sock, level, optname, optval, optlen); err =
sock->ops->getsockopt(sock, level, optname, optval,
optlen);
out_put: out_put:
fput_light(sock->file, fput_needed); fput_light(sock->file, fput_needed);
} }
return err; return err;
} }
/* /*
* Shutdown a socket. * Shutdown a socket.
*/ */
...@@ -1743,8 +1774,8 @@ asmlinkage long sys_shutdown(int fd, int how) ...@@ -1743,8 +1774,8 @@ asmlinkage long sys_shutdown(int fd, int how)
int err, fput_needed; int err, fput_needed;
struct socket *sock; struct socket *sock;
if ((sock = sockfd_lookup_light(fd, &err, &fput_needed))!=NULL) sock = sockfd_lookup_light(fd, &err, &fput_needed);
{ if (sock != NULL) {
err = security_socket_shutdown(sock, how); err = security_socket_shutdown(sock, how);
if (!err) if (!err)
err = sock->ops->shutdown(sock, how); err = sock->ops->shutdown(sock, how);
...@@ -1760,19 +1791,19 @@ asmlinkage long sys_shutdown(int fd, int how) ...@@ -1760,19 +1791,19 @@ asmlinkage long sys_shutdown(int fd, int how)
#define COMPAT_NAMELEN(msg) COMPAT_MSG(msg, msg_namelen) #define COMPAT_NAMELEN(msg) COMPAT_MSG(msg, msg_namelen)
#define COMPAT_FLAGS(msg) COMPAT_MSG(msg, msg_flags) #define COMPAT_FLAGS(msg) COMPAT_MSG(msg, msg_flags)
/* /*
* BSD sendmsg interface * BSD sendmsg interface
*/ */
asmlinkage long sys_sendmsg(int fd, struct msghdr __user *msg, unsigned flags) asmlinkage long sys_sendmsg(int fd, struct msghdr __user *msg, unsigned flags)
{ {
struct compat_msghdr __user *msg_compat = (struct compat_msghdr __user *)msg; struct compat_msghdr __user *msg_compat =
(struct compat_msghdr __user *)msg;
struct socket *sock; struct socket *sock;
char address[MAX_SOCK_ADDR]; char address[MAX_SOCK_ADDR];
struct iovec iovstack[UIO_FASTIOV], *iov = iovstack; struct iovec iovstack[UIO_FASTIOV], *iov = iovstack;
unsigned char ctl[sizeof(struct cmsghdr) + 20] unsigned char ctl[sizeof(struct cmsghdr) + 20]
__attribute__ ((aligned (sizeof(__kernel_size_t)))); __attribute__ ((aligned(sizeof(__kernel_size_t))));
/* 20 is size of ipv6_pktinfo */ /* 20 is size of ipv6_pktinfo */
unsigned char *ctl_buf = ctl; unsigned char *ctl_buf = ctl;
struct msghdr msg_sys; struct msghdr msg_sys;
...@@ -1783,7 +1814,8 @@ asmlinkage long sys_sendmsg(int fd, struct msghdr __user *msg, unsigned flags) ...@@ -1783,7 +1814,8 @@ asmlinkage long sys_sendmsg(int fd, struct msghdr __user *msg, unsigned flags)
if (MSG_CMSG_COMPAT & flags) { if (MSG_CMSG_COMPAT & flags) {
if (get_compat_msghdr(&msg_sys, msg_compat)) if (get_compat_msghdr(&msg_sys, msg_compat))
return -EFAULT; return -EFAULT;
} else if (copy_from_user(&msg_sys, msg, sizeof(struct msghdr))) }
else if (copy_from_user(&msg_sys, msg, sizeof(struct msghdr)))
return -EFAULT; return -EFAULT;
sock = sockfd_lookup_light(fd, &err, &fput_needed); sock = sockfd_lookup_light(fd, &err, &fput_needed);
...@@ -1795,7 +1827,7 @@ asmlinkage long sys_sendmsg(int fd, struct msghdr __user *msg, unsigned flags) ...@@ -1795,7 +1827,7 @@ asmlinkage long sys_sendmsg(int fd, struct msghdr __user *msg, unsigned flags)
if (msg_sys.msg_iovlen > UIO_MAXIOV) if (msg_sys.msg_iovlen > UIO_MAXIOV)
goto out_put; goto out_put;
/* Check whether to allocate the iovec area*/ /* Check whether to allocate the iovec area */
err = -ENOMEM; err = -ENOMEM;
iov_size = msg_sys.msg_iovlen * sizeof(struct iovec); iov_size = msg_sys.msg_iovlen * sizeof(struct iovec);
if (msg_sys.msg_iovlen > UIO_FASTIOV) { if (msg_sys.msg_iovlen > UIO_FASTIOV) {
...@@ -1819,14 +1851,15 @@ asmlinkage long sys_sendmsg(int fd, struct msghdr __user *msg, unsigned flags) ...@@ -1819,14 +1851,15 @@ asmlinkage long sys_sendmsg(int fd, struct msghdr __user *msg, unsigned flags)
goto out_freeiov; goto out_freeiov;
ctl_len = msg_sys.msg_controllen; ctl_len = msg_sys.msg_controllen;
if ((MSG_CMSG_COMPAT & flags) && ctl_len) { if ((MSG_CMSG_COMPAT & flags) && ctl_len) {
err = cmsghdr_from_user_compat_to_kern(&msg_sys, sock->sk, ctl, sizeof(ctl)); err =
cmsghdr_from_user_compat_to_kern(&msg_sys, sock->sk, ctl,
sizeof(ctl));
if (err) if (err)
goto out_freeiov; goto out_freeiov;
ctl_buf = msg_sys.msg_control; ctl_buf = msg_sys.msg_control;
ctl_len = msg_sys.msg_controllen; ctl_len = msg_sys.msg_controllen;
} else if (ctl_len) { } else if (ctl_len) {
if (ctl_len > sizeof(ctl)) if (ctl_len > sizeof(ctl)) {
{
ctl_buf = sock_kmalloc(sock->sk, ctl_len, GFP_KERNEL); ctl_buf = sock_kmalloc(sock->sk, ctl_len, GFP_KERNEL);
if (ctl_buf == NULL) if (ctl_buf == NULL)
goto out_freeiov; goto out_freeiov;
...@@ -1837,7 +1870,8 @@ asmlinkage long sys_sendmsg(int fd, struct msghdr __user *msg, unsigned flags) ...@@ -1837,7 +1870,8 @@ asmlinkage long sys_sendmsg(int fd, struct msghdr __user *msg, unsigned flags)
* Afterwards, it will be a kernel pointer. Thus the compiler-assisted * Afterwards, it will be a kernel pointer. Thus the compiler-assisted
* checking falls down on this. * checking falls down on this.
*/ */
if (copy_from_user(ctl_buf, (void __user *) msg_sys.msg_control, ctl_len)) if (copy_from_user(ctl_buf, (void __user *)msg_sys.msg_control,
ctl_len))
goto out_freectl; goto out_freectl;
msg_sys.msg_control = ctl_buf; msg_sys.msg_control = ctl_buf;
} }
...@@ -1863,12 +1897,14 @@ out: ...@@ -1863,12 +1897,14 @@ out:
* BSD recvmsg interface * BSD recvmsg interface
*/ */
asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flags) asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg,
unsigned int flags)
{ {
struct compat_msghdr __user *msg_compat = (struct compat_msghdr __user *)msg; struct compat_msghdr __user *msg_compat =
(struct compat_msghdr __user *)msg;
struct socket *sock; struct socket *sock;
struct iovec iovstack[UIO_FASTIOV]; struct iovec iovstack[UIO_FASTIOV];
struct iovec *iov=iovstack; struct iovec *iov = iovstack;
struct msghdr msg_sys; struct msghdr msg_sys;
unsigned long cmsg_ptr; unsigned long cmsg_ptr;
int err, iov_size, total_len, len; int err, iov_size, total_len, len;
...@@ -1884,8 +1920,8 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag ...@@ -1884,8 +1920,8 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag
if (MSG_CMSG_COMPAT & flags) { if (MSG_CMSG_COMPAT & flags) {
if (get_compat_msghdr(&msg_sys, msg_compat)) if (get_compat_msghdr(&msg_sys, msg_compat))
return -EFAULT; return -EFAULT;
} else }
if (copy_from_user(&msg_sys,msg,sizeof(struct msghdr))) else if (copy_from_user(&msg_sys, msg, sizeof(struct msghdr)))
return -EFAULT; return -EFAULT;
sock = sockfd_lookup_light(fd, &err, &fput_needed); sock = sockfd_lookup_light(fd, &err, &fput_needed);
...@@ -1896,7 +1932,7 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag ...@@ -1896,7 +1932,7 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag
if (msg_sys.msg_iovlen > UIO_MAXIOV) if (msg_sys.msg_iovlen > UIO_MAXIOV)
goto out_put; goto out_put;
/* Check whether to allocate the iovec area*/ /* Check whether to allocate the iovec area */
err = -ENOMEM; err = -ENOMEM;
iov_size = msg_sys.msg_iovlen * sizeof(struct iovec); iov_size = msg_sys.msg_iovlen * sizeof(struct iovec);
if (msg_sys.msg_iovlen > UIO_FASTIOV) { if (msg_sys.msg_iovlen > UIO_FASTIOV) {
...@@ -1910,7 +1946,7 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag ...@@ -1910,7 +1946,7 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag
* kernel msghdr to use the kernel address space) * kernel msghdr to use the kernel address space)
*/ */
uaddr = (void __user *) msg_sys.msg_name; uaddr = (void __user *)msg_sys.msg_name;
uaddr_len = COMPAT_NAMELEN(msg); uaddr_len = COMPAT_NAMELEN(msg);
if (MSG_CMSG_COMPAT & flags) { if (MSG_CMSG_COMPAT & flags) {
err = verify_compat_iovec(&msg_sys, iov, addr, VERIFY_WRITE); err = verify_compat_iovec(&msg_sys, iov, addr, VERIFY_WRITE);
...@@ -1918,7 +1954,7 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag ...@@ -1918,7 +1954,7 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag
err = verify_iovec(&msg_sys, iov, addr, VERIFY_WRITE); err = verify_iovec(&msg_sys, iov, addr, VERIFY_WRITE);
if (err < 0) if (err < 0)
goto out_freeiov; goto out_freeiov;
total_len=err; total_len = err;
cmsg_ptr = (unsigned long)msg_sys.msg_control; cmsg_ptr = (unsigned long)msg_sys.msg_control;
msg_sys.msg_flags = 0; msg_sys.msg_flags = 0;
...@@ -1933,7 +1969,8 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag ...@@ -1933,7 +1969,8 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag
len = err; len = err;
if (uaddr != NULL) { if (uaddr != NULL) {
err = move_addr_to_user(addr, msg_sys.msg_namelen, uaddr, uaddr_len); err = move_addr_to_user(addr, msg_sys.msg_namelen, uaddr,
uaddr_len);
if (err < 0) if (err < 0)
goto out_freeiov; goto out_freeiov;
} }
...@@ -1942,10 +1979,10 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag ...@@ -1942,10 +1979,10 @@ asmlinkage long sys_recvmsg(int fd, struct msghdr __user *msg, unsigned int flag
if (err) if (err)
goto out_freeiov; goto out_freeiov;
if (MSG_CMSG_COMPAT & flags) if (MSG_CMSG_COMPAT & flags)
err = __put_user((unsigned long)msg_sys.msg_control-cmsg_ptr, err = __put_user((unsigned long)msg_sys.msg_control - cmsg_ptr,
&msg_compat->msg_controllen); &msg_compat->msg_controllen);
else else
err = __put_user((unsigned long)msg_sys.msg_control-cmsg_ptr, err = __put_user((unsigned long)msg_sys.msg_control - cmsg_ptr,
&msg->msg_controllen); &msg->msg_controllen);
if (err) if (err)
goto out_freeiov; goto out_freeiov;
...@@ -1964,9 +2001,12 @@ out: ...@@ -1964,9 +2001,12 @@ out:
/* Argument list sizes for sys_socketcall */ /* Argument list sizes for sys_socketcall */
#define AL(x) ((x) * sizeof(unsigned long)) #define AL(x) ((x) * sizeof(unsigned long))
static unsigned char nargs[18]={AL(0),AL(3),AL(3),AL(3),AL(2),AL(3), static const unsigned char nargs[18]={
AL(0),AL(3),AL(3),AL(3),AL(2),AL(3),
AL(3),AL(3),AL(4),AL(4),AL(4),AL(6), AL(3),AL(3),AL(4),AL(4),AL(4),AL(6),
AL(6),AL(2),AL(5),AL(5),AL(3),AL(3)}; AL(6),AL(2),AL(5),AL(5),AL(3),AL(3)
};
#undef AL #undef AL
/* /*
...@@ -1980,54 +2020,59 @@ static unsigned char nargs[18]={AL(0),AL(3),AL(3),AL(3),AL(2),AL(3), ...@@ -1980,54 +2020,59 @@ static unsigned char nargs[18]={AL(0),AL(3),AL(3),AL(3),AL(2),AL(3),
asmlinkage long sys_socketcall(int call, unsigned long __user *args) asmlinkage long sys_socketcall(int call, unsigned long __user *args)
{ {
unsigned long a[6]; unsigned long a[6];
unsigned long a0,a1; unsigned long a0, a1;
int err; int err;
if(call<1||call>SYS_RECVMSG) if (call < 1 || call > SYS_RECVMSG)
return -EINVAL; return -EINVAL;
/* copy_from_user should be SMP safe. */ /* copy_from_user should be SMP safe. */
if (copy_from_user(a, args, nargs[call])) if (copy_from_user(a, args, nargs[call]))
return -EFAULT; return -EFAULT;
err = audit_socketcall(nargs[call]/sizeof(unsigned long), a); err = audit_socketcall(nargs[call] / sizeof(unsigned long), a);
if (err) if (err)
return err; return err;
a0=a[0]; a0 = a[0];
a1=a[1]; a1 = a[1];
switch(call) switch (call) {
{
case SYS_SOCKET: case SYS_SOCKET:
err = sys_socket(a0,a1,a[2]); err = sys_socket(a0, a1, a[2]);
break; break;
case SYS_BIND: case SYS_BIND:
err = sys_bind(a0,(struct sockaddr __user *)a1, a[2]); err = sys_bind(a0, (struct sockaddr __user *)a1, a[2]);
break; break;
case SYS_CONNECT: case SYS_CONNECT:
err = sys_connect(a0, (struct sockaddr __user *)a1, a[2]); err = sys_connect(a0, (struct sockaddr __user *)a1, a[2]);
break; break;
case SYS_LISTEN: case SYS_LISTEN:
err = sys_listen(a0,a1); err = sys_listen(a0, a1);
break; break;
case SYS_ACCEPT: case SYS_ACCEPT:
err = sys_accept(a0,(struct sockaddr __user *)a1, (int __user *)a[2]); err =
sys_accept(a0, (struct sockaddr __user *)a1,
(int __user *)a[2]);
break; break;
case SYS_GETSOCKNAME: case SYS_GETSOCKNAME:
err = sys_getsockname(a0,(struct sockaddr __user *)a1, (int __user *)a[2]); err =
sys_getsockname(a0, (struct sockaddr __user *)a1,
(int __user *)a[2]);
break; break;
case SYS_GETPEERNAME: case SYS_GETPEERNAME:
err = sys_getpeername(a0, (struct sockaddr __user *)a1, (int __user *)a[2]); err =
sys_getpeername(a0, (struct sockaddr __user *)a1,
(int __user *)a[2]);
break; break;
case SYS_SOCKETPAIR: case SYS_SOCKETPAIR:
err = sys_socketpair(a0,a1, a[2], (int __user *)a[3]); err = sys_socketpair(a0, a1, a[2], (int __user *)a[3]);
break; break;
case SYS_SEND: case SYS_SEND:
err = sys_send(a0, (void __user *)a1, a[2], a[3]); err = sys_send(a0, (void __user *)a1, a[2], a[3]);
break; break;
case SYS_SENDTO: case SYS_SENDTO:
err = sys_sendto(a0,(void __user *)a1, a[2], a[3], err = sys_sendto(a0, (void __user *)a1, a[2], a[3],
(struct sockaddr __user *)a[4], a[5]); (struct sockaddr __user *)a[4], a[5]);
break; break;
case SYS_RECV: case SYS_RECV:
...@@ -2035,22 +2080,25 @@ asmlinkage long sys_socketcall(int call, unsigned long __user *args) ...@@ -2035,22 +2080,25 @@ asmlinkage long sys_socketcall(int call, unsigned long __user *args)
break; break;
case SYS_RECVFROM: case SYS_RECVFROM:
err = sys_recvfrom(a0, (void __user *)a1, a[2], a[3], err = sys_recvfrom(a0, (void __user *)a1, a[2], a[3],
(struct sockaddr __user *)a[4], (int __user *)a[5]); (struct sockaddr __user *)a[4],
(int __user *)a[5]);
break; break;
case SYS_SHUTDOWN: case SYS_SHUTDOWN:
err = sys_shutdown(a0,a1); err = sys_shutdown(a0, a1);
break; break;
case SYS_SETSOCKOPT: case SYS_SETSOCKOPT:
err = sys_setsockopt(a0, a1, a[2], (char __user *)a[3], a[4]); err = sys_setsockopt(a0, a1, a[2], (char __user *)a[3], a[4]);
break; break;
case SYS_GETSOCKOPT: case SYS_GETSOCKOPT:
err = sys_getsockopt(a0, a1, a[2], (char __user *)a[3], (int __user *)a[4]); err =
sys_getsockopt(a0, a1, a[2], (char __user *)a[3],
(int __user *)a[4]);
break; break;
case SYS_SENDMSG: case SYS_SENDMSG:
err = sys_sendmsg(a0, (struct msghdr __user *) a1, a[2]); err = sys_sendmsg(a0, (struct msghdr __user *)a1, a[2]);
break; break;
case SYS_RECVMSG: case SYS_RECVMSG:
err = sys_recvmsg(a0, (struct msghdr __user *) a1, a[2]); err = sys_recvmsg(a0, (struct msghdr __user *)a1, a[2]);
break; break;
default: default:
err = -EINVAL; err = -EINVAL;
...@@ -2072,18 +2120,18 @@ int sock_register(struct net_proto_family *ops) ...@@ -2072,18 +2120,18 @@ int sock_register(struct net_proto_family *ops)
int err; int err;
if (ops->family >= NPROTO) { if (ops->family >= NPROTO) {
printk(KERN_CRIT "protocol %d >= NPROTO(%d)\n", ops->family, NPROTO); printk(KERN_CRIT "protocol %d >= NPROTO(%d)\n", ops->family,
NPROTO);
return -ENOBUFS; return -ENOBUFS;
} }
net_family_write_lock(); net_family_write_lock();
err = -EEXIST; err = -EEXIST;
if (net_families[ops->family] == NULL) { if (net_families[ops->family] == NULL) {
net_families[ops->family]=ops; net_families[ops->family] = ops;
err = 0; err = 0;
} }
net_family_write_unlock(); net_family_write_unlock();
printk(KERN_INFO "NET: Registered protocol family %d\n", printk(KERN_INFO "NET: Registered protocol family %d\n", ops->family);
ops->family);
return err; return err;
} }
...@@ -2099,10 +2147,9 @@ int sock_unregister(int family) ...@@ -2099,10 +2147,9 @@ int sock_unregister(int family)
return -1; return -1;
net_family_write_lock(); net_family_write_lock();
net_families[family]=NULL; net_families[family] = NULL;
net_family_write_unlock(); net_family_write_unlock();
printk(KERN_INFO "NET: Unregistered protocol family %d\n", printk(KERN_INFO "NET: Unregistered protocol family %d\n", family);
family);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment