diff options
| -rw-r--r-- | net/socket.c | 18 |
1 files changed, 15 insertions, 3 deletions
diff --git a/net/socket.c b/net/socket.c index af57d85bcb48..8a109012608a 100644 --- a/net/socket.c +++ b/net/socket.c | |||
| @@ -541,7 +541,10 @@ static int sockfs_setattr(struct dentry *dentry, struct iattr *iattr) | |||
| 541 | if (!err && (iattr->ia_valid & ATTR_UID)) { | 541 | if (!err && (iattr->ia_valid & ATTR_UID)) { |
| 542 | struct socket *sock = SOCKET_I(d_inode(dentry)); | 542 | struct socket *sock = SOCKET_I(d_inode(dentry)); |
| 543 | 543 | ||
| 544 | sock->sk->sk_uid = iattr->ia_uid; | 544 | if (sock->sk) |
| 545 | sock->sk->sk_uid = iattr->ia_uid; | ||
| 546 | else | ||
| 547 | err = -ENOENT; | ||
| 545 | } | 548 | } |
| 546 | 549 | ||
| 547 | return err; | 550 | return err; |
| @@ -590,12 +593,16 @@ EXPORT_SYMBOL(sock_alloc); | |||
| 590 | * an inode not a file. | 593 | * an inode not a file. |
| 591 | */ | 594 | */ |
| 592 | 595 | ||
| 593 | void sock_release(struct socket *sock) | 596 | static void __sock_release(struct socket *sock, struct inode *inode) |
| 594 | { | 597 | { |
| 595 | if (sock->ops) { | 598 | if (sock->ops) { |
| 596 | struct module *owner = sock->ops->owner; | 599 | struct module *owner = sock->ops->owner; |
| 597 | 600 | ||
| 601 | if (inode) | ||
| 602 | inode_lock(inode); | ||
| 598 | sock->ops->release(sock); | 603 | sock->ops->release(sock); |
| 604 | if (inode) | ||
| 605 | inode_unlock(inode); | ||
| 599 | sock->ops = NULL; | 606 | sock->ops = NULL; |
| 600 | module_put(owner); | 607 | module_put(owner); |
| 601 | } | 608 | } |
| @@ -609,6 +616,11 @@ void sock_release(struct socket *sock) | |||
| 609 | } | 616 | } |
| 610 | sock->file = NULL; | 617 | sock->file = NULL; |
| 611 | } | 618 | } |
| 619 | |||
| 620 | void sock_release(struct socket *sock) | ||
| 621 | { | ||
| 622 | __sock_release(sock, NULL); | ||
| 623 | } | ||
| 612 | EXPORT_SYMBOL(sock_release); | 624 | EXPORT_SYMBOL(sock_release); |
| 613 | 625 | ||
| 614 | void __sock_tx_timestamp(__u16 tsflags, __u8 *tx_flags) | 626 | void __sock_tx_timestamp(__u16 tsflags, __u8 *tx_flags) |
| @@ -1171,7 +1183,7 @@ static int sock_mmap(struct file *file, struct vm_area_struct *vma) | |||
| 1171 | 1183 | ||
| 1172 | static int sock_close(struct inode *inode, struct file *filp) | 1184 | static int sock_close(struct inode *inode, struct file *filp) |
| 1173 | { | 1185 | { |
| 1174 | sock_release(SOCKET_I(inode)); | 1186 | __sock_release(SOCKET_I(inode), inode); |
| 1175 | return 0; | 1187 | return 0; |
| 1176 | } | 1188 | } |
| 1177 | 1189 | ||
