tech-kern archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: [PATCH] POSIX extended API set 2
David Holland <dholland-tech%netbsd.org@localhost> wrote:
> > In order to check for absolute path, I need to copyin the path, and it
> > wil be copied in later again. I suspect that check should go at namei
> > level.
>
> I don't understand. Why do you need to copy in the path twice? Don't
> do that.
Yes, I checked your namei code in the meantime, and the absolute path in
indeed checked for. It is just sad we still do the fd_getfile/fd_putfile
for nothing if it is an absolute path.
In fact I think it would make simplier and less error-prone code if NDAT
would take a file descriptor instead of a vnode. fd_getfile/fd_putfile
would be done only once in our codebase in namei(). I understand it is
not be desirable regarding how the kernel is layered, though.
--
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
manu%netbsd.org@localhost
Home |
Main Index |
Thread Index |
Old Index