Josh Triplett <josh@xxxxxxxxxxxxxxxx> writes: > On November 15, 2014 6:05:11 PM PST, Theodore Ts'o <tytso@xxxxxxx> wrote: >>On Sat, Nov 15, 2014 at 12:20:42PM -0800, Josh Triplett wrote: >>> > However, sudoers seems to allow negative group matches. So maybe >>> > allowing this only with no_new_privs already set would make sense. >>> >>> Sigh, bad sudo. Sure, restricting this to no_new_privs only seems >>fine. >>> I'll do that in v2, and document that in the manpage. >> >>I've also seen use cases (generally back in the bad old days of big >>timesharing VAX 750's :-) where the system admin might assign someone >>to the "games-abusers" group, and then set /usr/games to mode 705 >>root:games-abusers --- presumably because it's easier to add a few >>people to the deny list rather than having to add all of the EECS >>department to the games group minus the abusers. >> >>So arbitrarily anyone to drop groups from their supplemental group >>list will result in a change from both existing practice and legacy >>Unix systems, and it could potentially lead to a security exposure. > > As Andy pointed out, you can already do that with a user namespace, That may be a bug with the user namespace permission check. Perhaps we shouldn't allow dropping groups that aren't mapped in the user namespace. To add to the discussion there are also sg and newgroup, though I don't think they touch supplementary groups. > for any case not involving a setuid or setgid (or otherwise > privilege-gaining) program. And requiring no_new_privs handles that. Frankly adding a no_new_privs check to allow something/anything scares me. That converts no_new_privs from something simple and easy to understand to something much more complicated. > Given the combination of those two things, do you still see any > problematic cases? Josh I think it is time to stop and make certain you understand how unix groups work in the large. It seems to me that either supplementary groups can be dropped or supplementary groups can not be dropped. If they can be dropped we shouldn't need complicated checks to allow them to be dropped in some circumstances but not others. Unix groups are an old interface and they have been used in a lot of ways over the years. We need to be careful any change we make is good and truly backwards compatible and that we do our darndest not to introduce security holes. Eric -- To unsubscribe from this list: send the line "unsubscribe linux-api" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
