lkml.org 
[lkml]   [2007]   [Jan]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: mprotect abuse in slim
    Quoting Christoph Hellwig (hch@infradead.org):
    > On Mon, Jan 08, 2007 at 07:07:25PM -0800, Arjan van de Ven wrote:
    > >
    > > > Starting with the fdtable, would it help if we move the
    > > > fdtable tweaking out of slim itself and into helpers? Or
    > > > can you recommend another way to implement this functionality.
    > >
    > > Hi,
    > >
    > > maybe this is a silly question, but do you revoke not only the current
    > > fd entries, but also the ones that are pending in UNIX domain sockets
    > > and that are already being sent to the process? If not.. then you might
    > > as well not bother ;)
    >
    > Exactly. What these folks want is revoke (maybe more fine grained, but
    > that's not the point). And guess what folks, revoke is not trivial,
    > otherwise we'd have it. If you want to volunteer to implement a full-blown
    > revoke that's fine, but
    >
    > a) it belongs into core code
    > b) needs to be done right

    Whatever happened with Pekka's revoke submissions? Did you lose
    interest after
    http://www.kernel.org/pub/linux/kernel/people/penberg/patches/revoke/2.6.19-rc1/revoke-2.6.19-rc1,
    or was it decided that the approach was unworkable?

    Now, what slim needs isn't "revoke all files for this inode",
    but "revoke this task's write access to this fd". So two functions
    which could be useful are

    int fd_revoke_write(struct task_struct *tsk, int fd)
    int fd_revoke_write_iter(struct task_struct *tsk,
    (int *)need_revoke(struct task_struct *tsk, int fd))

    Anyway I'll get going on rebasing Pekka's latest patch (pending
    his reply) and providing the above two functions on top of that.

    -serge
    -
    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-01-10 00:53    [W:0.021 / U:30.500 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site