Re: [PATCH] net: socket: don't set sk_uid to garbage value in ->setattr()

2017-01-01 Thread David Miller
From: Lorenzo Colitti Date: Sun, 1 Jan 2017 16:57:23 +0900 > On Sat, Dec 31, 2016 at 8:42 AM, Eric Biggers wrote: >> ->setattr() was recently implemented for socket files to sync the socket >> inode's uid to the new 'sk_uid' member of struct sock. It

Re: [PATCH] net: socket: don't set sk_uid to garbage value in ->setattr()

2017-01-01 Thread David Miller
From: Lorenzo Colitti Date: Sun, 1 Jan 2017 16:57:23 +0900 > On Sat, Dec 31, 2016 at 8:42 AM, Eric Biggers wrote: >> ->setattr() was recently implemented for socket files to sync the socket >> inode's uid to the new 'sk_uid' member of struct sock. It does this by >> copying over the ia_uid

Re: [PATCH] net: socket: don't set sk_uid to garbage value in ->setattr()

2016-12-31 Thread Lorenzo Colitti
On Sat, Dec 31, 2016 at 8:42 AM, Eric Biggers wrote: > ->setattr() was recently implemented for socket files to sync the socket > inode's uid to the new 'sk_uid' member of struct sock. It does this by > copying over the ia_uid member of struct iattr. However, ia_uid is >

Re: [PATCH] net: socket: don't set sk_uid to garbage value in ->setattr()

2016-12-31 Thread Lorenzo Colitti
On Sat, Dec 31, 2016 at 8:42 AM, Eric Biggers wrote: > ->setattr() was recently implemented for socket files to sync the socket > inode's uid to the new 'sk_uid' member of struct sock. It does this by > copying over the ia_uid member of struct iattr. However, ia_uid is > actually only valid

Re: [PATCH] net: socket: don't set sk_uid to garbage value in ->setattr()

2016-12-31 Thread David Miller
From: Eric Biggers Date: Fri, 30 Dec 2016 17:42:32 -0600 > From: Eric Biggers > > ->setattr() was recently implemented for socket files to sync the socket > inode's uid to the new 'sk_uid' member of struct sock. It does this by > copying over the

Re: [PATCH] net: socket: don't set sk_uid to garbage value in ->setattr()

2016-12-31 Thread David Miller
From: Eric Biggers Date: Fri, 30 Dec 2016 17:42:32 -0600 > From: Eric Biggers > > ->setattr() was recently implemented for socket files to sync the socket > inode's uid to the new 'sk_uid' member of struct sock. It does this by > copying over the ia_uid member of struct iattr. However,

[PATCH] net: socket: don't set sk_uid to garbage value in ->setattr()

2016-12-30 Thread Eric Biggers
From: Eric Biggers ->setattr() was recently implemented for socket files to sync the socket inode's uid to the new 'sk_uid' member of struct sock. It does this by copying over the ia_uid member of struct iattr. However, ia_uid is actually only valid when ATTR_UID is set in

[PATCH] net: socket: don't set sk_uid to garbage value in ->setattr()

2016-12-30 Thread Eric Biggers
From: Eric Biggers ->setattr() was recently implemented for socket files to sync the socket inode's uid to the new 'sk_uid' member of struct sock. It does this by copying over the ia_uid member of struct iattr. However, ia_uid is actually only valid when ATTR_UID is set in ia_valid, indicating