All of lore.kernel.org
 help / color / mirror / Atom feed
From: Karl MacMillan <kmacmillan@mentalrootkit.com>
To: Joshua Brindle <jbrindle@tresys.com>
Cc: Eric Paris <eparis@parisplace.org>,
	selinux@tycho.nsa.gov, sds@tycho.nsa.gov, jmorris@namei.org
Subject: Re: [PATCH] SELinux: new /proc/self/attr/ipccreate for explicite ipc	object labeling
Date: Wed, 19 Jul 2006 10:42:17 -0400	[thread overview]
Message-ID: <1153320137.2864.0.camel@localhost.localdomain> (raw)
In-Reply-To: <44BE402E.4020407@tresys.com>

On Wed, 2006-07-19 at 10:22 -0400, Joshua Brindle wrote:
> Eric Paris wrote:
> > At this point in time IPC objects (semaphores, message queues, etc) were
> > labeled with the label of the process which created them.  This patch
> > introduces a new /proc/self/attr/ipccreate which will allow a program to
> > set the label on ipc objects it is about to create.  This may be useful
> > in future trusted applications but at this time I do not know of any
> > application which needs this functionality, but it should allow better
> > more understandable policy to control access to ipc objects.
> >
> > In security/selinux/include/av_permissions.h the patch also adds a blank
> > line at the end.  This was the output after the change to the
> > access_vectors in policy and I did not edit that file by hand.
> >
> >   
> This patch only adds a process permission for setipccreate, I think you 
> need a permission check for the destination type (what you are labeling 
> the ipc to), there are lots of potential info leaks even if the process 
> can't subsequently write to the ipc.
> 

This is covered by the existing create permissions on the IPC objects
since this is not a relabel but object creation.

Karl

> --
> 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.
> 

--
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-07-19 14:42 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-07-18 19:36 [PATCH] SELinux: new /proc/self/attr/ipccreate for explicite ipc object labeling Eric Paris
2006-07-18 20:01 ` Karl MacMillan
2006-07-18 20:05 ` Stephen Smalley
2006-07-19 14:22 ` Joshua Brindle
2006-07-19 14:42   ` Karl MacMillan [this message]
2006-07-19 14:59     ` [PATCH] SELinux: new /proc/self/attr/ipccreate for expliciteipc " Joshua Brindle
2006-07-27 16:58       ` Eric Paris
2006-07-27 18:11         ` Stephen Smalley
2006-07-27 21:16         ` Valdis.Kletnieks
2006-07-28  4:17           ` James Morris
2006-07-28 11:42             ` Stephen Smalley
2006-07-28 14:12           ` Eric Paris
2006-07-28  4:19         ` James Morris
2006-07-28  5:23           ` Russell Coker

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=1153320137.2864.0.camel@localhost.localdomain \
    --to=kmacmillan@mentalrootkit.com \
    --cc=eparis@parisplace.org \
    --cc=jbrindle@tresys.com \
    --cc=jmorris@namei.org \
    --cc=sds@tycho.nsa.gov \
    --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.