All of lore.kernel.org
 help / color / mirror / Atom feed
From: Russell Coker <russell@coker.com.au>
To: Thomas Bleher <bleher@informatik.uni-muenchen.de>
Cc: SELinux ML <selinux@tycho.nsa.gov>
Subject: Re: file_contexts patch
Date: Sat, 17 Jul 2004 11:59:00 +1000	[thread overview]
Message-ID: <200407171159.00489.russell@coker.com.au> (raw)
In-Reply-To: <20040716224252.GD4940@obadja.jmh.mhn.de>

On Sat, 17 Jul 2004 08:42, Thomas Bleher <bleher@informatik.uni-muenchen.de> 
wrote:
> > Why does kdesktop_lock need to be labelled as screensaver_exec_t?  The
> > problem that caused the creation of the screen saver policy has only been
> > reported under GNOME, and kdesktop_lock has been working well for me for
> > years without such policy.
>
> The screensaver policy has auth_chkpwd which is needed when locking the
> screen IIRC. The only problem is that kdesktop_lock doesn't use
> unix_chkpwd but has its own program (kcheckpass). I have labeled it
> chkpwd_exec_t which works fine, but unfortunately kcheckpass needs some
> additional permissions (from macros/program/chkpwd_macros.te):

kcheckpass should use PAM and therefore use unix_chkpwd.

> ifelse($1, `system', `', `allow $1_chkpwd_t $1_t:unix_stream_socket { read
> write };') # read KDesktop
> allow system_chkpwd_t $1_home_t:dir read;
> # write to .xsession-errors
> allow system_chkpwd_t $1_home_t:file write;
> (Or something like that. I live on NFS and just converted the rules)
>
> I do not know if this is mergable into mainline.

It's not something that we want to do if we can avoid it.

> > What is /dev/usbscanner?  It's not in devices.txt in the kernel source
> > and not installed by default on any Debian or Fedora system.
>
> Call it a SuSE oddity:
> # ls -l /dev/usbscanner /dev/usb/scanner0
> crw-rw----    1 root     root     180,  48 Sep  9  2002 /dev/usb/scanner0
> crw-r--r--    1 root     root     180,  48 Sep  9  2002 /dev/usbscanner
> (why they have different permissions there I can't imagine)

Could you file a bug report in the SUSE bug tracking system about this?

I suggest that you request that /dev/usbscanner become a sym-link 
to /dev/usb/scanner0.  Then it will match the devices.txt in the kernel 
source, have no problems with consistency of permissions, and not need any 
modifications to the SE Linux policy.

Maybe we should have a suse.te and suse.fc for such things?  One thing we want 
to avoid if possible is having entries in types.fc for distribution specific 
names which bloat types.fc and make setfiles run more slowly.

-- 
http://www.coker.com.au/selinux/   My NSA Security Enhanced Linux packages
http://www.coker.com.au/bonnie++/  Bonnie++ hard drive benchmark
http://www.coker.com.au/postal/    Postal SMTP/POP benchmark
http://www.coker.com.au/~russell/  My home page

--
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:[~2004-07-17  1:59 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-07-16 20:15 file_contexts patch Thomas Bleher
2004-07-16 22:02 ` Russell Coker
2004-07-16 22:42   ` Thomas Bleher
2004-07-17  1:59     ` Russell Coker [this message]
2004-07-17 19:53       ` Thomas Bleher
2004-07-18  2:58         ` Russell Coker
2004-07-18 15:03           ` Thomas Bleher
2004-07-18 23:59             ` Russell Coker
2004-07-24 19:08               ` Thomas Bleher
2004-07-26 20:25         ` Stephen Smalley
2004-07-26 22:06           ` Thomas Bleher
2004-07-28  6:48             ` Russell Coker
2004-07-28 19:25             ` Stephen Smalley
2004-07-27  0:36           ` Valdis.Kletnieks
2004-07-27 13:38           ` Stephen Smalley
2004-07-27 17:17             ` Valdis.Kletnieks
2004-07-28  2:53             ` Russell Coker
2004-07-30 13:33               ` Stephen Smalley
2004-07-17 10:15 ` Luke Kenneth Casson Leighton
2004-07-17 13:52   ` Thomas Bleher

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=200407171159.00489.russell@coker.com.au \
    --to=russell@coker.com.au \
    --cc=bleher@informatik.uni-muenchen.de \
    --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.