lkml.org 
[lkml]   [2007]   [Apr]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [patch 3/8] account user mounts
    From
    Date
    > > From: Miklos Szeredi <mszeredi@suse.cz>
    > >
    > > Add sysctl variables for accounting and limiting the number of user
    > > mounts.
    > >
    > > The maximum number of user mounts is set to 1024 by default. This
    > > won't in itself enable user mounts, setting a mount to be owned by a
    > > user is first needed
    >
    > Since each mount has a user can we just make this a per user rlimit?
    >
    > If we are going to implement a sysctl at this point I think it should
    > be a global limit that doesn't care if who you are. Even root can
    > have recursive mounts that attempt to get out of control.

    Recursive bind mounts are done carefully enough, so they don't get out
    of control.

    Recursive mount propagations can get out of control. But root can
    shoot itself in the foot any number of ways, and it's not for the
    kernel to police that.

    > Also currently you are not checking the max_users. It looks like
    > you do this in a later patch but still it is a little strange to
    > allow user own mounts and have accounting but to not check the
    > limit at this state.

    Yeah, but at this stage user mounts are not yet allowed, so this is
    safe.

    Miklos
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-04-22 09:15    [W:0.020 / U:58.688 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site