Re: [PATCH 14/13] sysfs: sysfs_setattr remove unnecessary permissioncheck.

From: Tejun Heo
Date: Fri Nov 06 2009 - 21:18:53 EST

Eric W. Biederman wrote:
> inode_change_ok already clears the SGID bit when necessary so there is
> no reason for sysfs_setattr to carry code to do the same, and it is
> good to kill the extra copy because when I moved the code, I goofed
> and in certain corner cases the code will look at the wrong gid.
> Signed-off-by: Eric W. Biederman <ebiederm@xxxxxxxxxxxxxxxxxx>

Acked-by: Tejun Heo <tj@xxxxxxxxxx>

I would much prefer this one being prior to 06 but if it's too
painful, please at least note it in the description of 06.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at