All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ivan Gyurdiev <ivg2@cornell.edu>
To: Joshua Brindle <jbrindle@tresys.com>
Cc: Daniel J Walsh <dwalsh@redhat.com>, SE Linux <selinux@tycho.nsa.gov>
Subject: Re: Why are we managing seusers file via libsemanage?
Date: Tue, 17 Jan 2006 00:36:13 -0700	[thread overview]
Message-ID: <43CC9E6D.9090509@cornell.edu> (raw)
In-Reply-To: <43CC81C2.5010104@tresys.com>

>
> To clarify: The library needs to do the validation no matter what. The 
> policy isn't exposed to any userland tools so semanage can't do 
> checking itself. 

Not entirely true - the client could query the necessary information via 
semanage, and do validation itself. You can take the entire 
seuser_validate function, take it out of semanage, change it a little 
bit, and put put in a transaction, and it would work just fine.

> So whether the semanage tool writes the file or libsemanage writes the 
> file the library does validation, I don't see a compelling reason to 
> push this parsing/writing code to the client.
My reasoning is - whether the info is stored in LDAP or not is an 
implementation detail. It should not affect the data model that we're 
working with. It makes sense to put seusers in libsemanage, because that 
allows management of all selinux-related things from a single shared 
library in a consistent interface, with shared transactions and 
validation. It's also a good point for integration with the policy server.

In other words, libsemanage should pull the data out of LDAP, hiding the 
implementation details. I'm not sure how it would do that exactly - 
dlopen another library, call outside tool, will have to look into that. 
I've tried to make the libsemanage part of it flexible, to deal with 
this issue eventually. However, I think libsemanage should be the 
management frontend. Letting the storage backend determine interface 
structure seems like the wrong approach - either the interface makes 
sense, or it doesn't - why should we change it, if the info is backed in 
LDAP?


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

  reply	other threads:[~2006-01-17  7:36 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-17  3:49 Why are we managing seusers file via libsemanage? Daniel J Walsh
2006-01-17  5:27 ` Joshua Brindle
2006-01-17  5:33   ` Joshua Brindle
2006-01-17  7:36     ` Ivan Gyurdiev [this message]
2006-01-17  8:10       ` Ivan Gyurdiev
2006-01-17  7:16   ` Ivan Gyurdiev
2006-01-17  7:57 ` Ivan Gyurdiev
2006-01-17 18:02   ` Daniel J Walsh
2006-01-17 18:39     ` Ivan Gyurdiev
  -- strict thread matches above, loose matches on Subject: below --
2006-01-17 16:56 schaufler-ca.com - Casey Schaufler
2006-01-17 19:02 ` Ivan Gyurdiev

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=43CC9E6D.9090509@cornell.edu \
    --to=ivg2@cornell.edu \
    --cc=dwalsh@redhat.com \
    --cc=jbrindle@tresys.com \
    --cc=selinux@tycho.nsa.gov \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.