All of lore.kernel.org
 help / color / mirror / Atom feed
From: sven.vermeulen@siphos.be (Sven Vermeulen)
To: refpolicy@oss.tresys.com
Subject: [refpolicy] [REVIEW REQUEST] Changes to the pulseaudio policy module and its dependencies
Date: Fri, 19 Oct 2012 20:45:11 +0200	[thread overview]
Message-ID: <20121019184511.GA14349@siphos.be> (raw)
In-Reply-To: <1350670399.12496.24.camel@d30.localdomain>

On Fri, Oct 19, 2012 at 08:13:19PM +0200, Dominick Grift wrote:
> > I have a similar construction with alsa. One thing I am hoping to look into
> > soon is a "What if /dev/shm was shm_tmpfs_t instead of tmpfs_t", would that
> > make sense?
> > 
> > It would tighten the scope of such "wide" tmpfs file accesses.
> 
> pulseaudio clients create that pulse-shm file in /dev/shm with a private
> type, i do not see how this would help?
> 
> programs that use pulse and that run in the userdomain create those
> files with user_tmpfs_t
> 
> So all they need with regard to tmpfs_t is search tmpfs_t dirs, add and
> remove dir entries from tmpfs_t dirs and get attributes of tmpfs_t
> filesystems

It's mainly for reducing the scope. If pulseaudio clients automatically have
their tmpfs types be writeable by other pulseaudio clients, then also
non-shm related tmpfs files are affected by this.

If we'd use a specific type for shared memory (like shm_t) then other tmpfs
files are not "opened up".

It's a fairly intrusive change for a small gain, I understand (many policies
will need to be adapted for this, whereas there aren't that many tmpfs
locations on a system I think that are actually labeled as tmpfs_t). Hence
the "look into" part of the suggestion ;)

Wkr,
	Sven Vermeulen

  reply	other threads:[~2012-10-19 18:45 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-19 17:23 [refpolicy] [REVIEW REQUEST] Changes to the pulseaudio policy module and its dependencies Dominick Grift
2012-10-19 17:31 ` Dominick Grift
2012-10-19 18:00 ` Sven Vermeulen
2012-10-19 18:11   ` Daniel J Walsh
2012-10-19 18:13   ` Dominick Grift
2012-10-19 18:45     ` Sven Vermeulen [this message]
2012-10-19 20:51     ` Daniel J Walsh
2012-10-19 21:03       ` Dominick Grift
2012-10-29 12:42 ` Dominick Grift

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=20121019184511.GA14349@siphos.be \
    --to=sven.vermeulen@siphos.be \
    --cc=refpolicy@oss.tresys.com \
    /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.