--- Stephen Smalley <sds@xxxxxxxxxxxxx> wrote: > On Thu, 2007-08-09 at 10:07 -0700, Casey Schaufler wrote: > > --- David Howells <dhowells@xxxxxxxxxx> wrote: > > > > > Permit an inode's security ID to be obtained by the CacheFiles module. > This > > > is > > > then used as the SID with which files and directories will be created in > the > > > cache. > > > > This is SELinux specific functionality. It should not be an LSM > > interface. > > Odd, you proposed exactly the same hook (aside from naming convention > and secid as argument vs. as retval) in recent postings on linux-audit > and selinux list for use by the audit system. And that's exposing SELinux specific functionality too. And I don't like the fact that the audit system already requires a secid interface. The audit system, however, does not use the secid for anything other than a handle that gets passed around and eventually used to get the data that goes into the audit record. It's annoying, but harmless and does not affect any access control decisions. The change proposed here would use the secid in access control decisions. The LSM interface ought not to be exposing module specific internal data structures. My work on pulling selinux code out of audit left the secid interface in place. You're right in that audit should get fixed. I had been hoping to make that a phase II activity. Casey Schaufler casey@xxxxxxxxxxxxxxxx - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html