Re: mprotect abuse in slim

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Christoph Hellwig ([email protected]):
> 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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux