Re: [PATCH 2/4] integrity: Linux Integrity Module(LIM)

[Christoph Hellwig]

On Wed, Nov 12, 2008 at 10:47:12PM -0500, Mimi Zohar wrote:
> This version resolves the merge issues resulting from the removal
> of the nameidata parameter to inode_permission(), by moving the
> integrity_inode_permission() call from inode_permission() to
> may_open(), and renaming the hook to integrity_nameidata_check().

Still neither a good parameter nor a good name.  If you look at fsdevel
you'll see that a patch for the next merged window is already out and
in Al's to be applied queue that removes the nameidata there.  A
strcut path and maybe the MAY_ lookup flags should be enough for you,
and a integrity_path_check also sounds like it really does what you
want.

Also please merge the last radix tree patch into this one so tat it's
possible to review the code that actually goes in instead of replacing
a large part of it in a follow-up patch.

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