All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael C Thompson <thompsmc@us.ibm.com>
To: Daniel J Walsh <dwalsh@redhat.com>
Cc: "Christopher J. PeBenito" <cpebenito@tresys.com>,
	SE Linux <selinux@tycho.nsa.gov>
Subject: Re: Samba fixes
Date: Thu, 19 Apr 2007 13:32:32 -0500	[thread overview]
Message-ID: <4627B5C0.7070305@us.ibm.com> (raw)
In-Reply-To: <4627AC90.2050502@redhat.com>

Daniel J Walsh wrote:
> Michael C Thompson wrote:
>> Daniel J Walsh wrote:
>>> I have been working heavily with the samba team here to get selinux 
>>> policy working well with samba.
>>>
>>>
>>> Most controversial part is
>>> samba_unconfined_script_exec_t
>>>
>>> Which is a directory that administrators can put random scripts into 
>>> and allow samba to execute.
>>
>> So, three questions:
>> 1) What user would be executing these scripts? The 'samba' user?
> Samba Developers say:
> 
> Either the authenticated user or root, it really depends on the script.
> 
> For example, a "preexec" script is run before the share is accessed as
> the user that authenticated to samba (or the forced user as per "force
> user" parameter), while a "root preexec" script would always be run as
> root. Other scripts always run as root or always as auth user depending
> on the action to be performed (ie add user scripts always run as root,
> while print related stuff should always run as user).
> 
>>
>> 2) What is the intention of such functionality? To have samba be able 
>> to run file management tools or something?
>> 3) Is supporting this functionality even a good idea?
> As opposed to setenforce 0/or samba_disable_trans?

That might be better than running with a false sense of security :) Even 
though this is an "administrative restricted" set of scripts, if an 
attacker could place malicious scripts in the directory, then permitting 
samba to exec them as root could be an avenue of attack.


My original point was simply that if its so controversial, that might 
beg the question "is offering this functionality a good idea, in the 
first place?".

Mike

--
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:[~2007-04-19 18:32 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-19 14:28 Samba fixes Daniel J Walsh
2007-04-19 15:15 ` Michael C Thompson
2007-04-19 17:53   ` Daniel J Walsh
2007-04-19 18:32     ` Michael C Thompson [this message]
2007-04-19 19:11       ` Karl MacMillan

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=4627B5C0.7070305@us.ibm.com \
    --to=thompsmc@us.ibm.com \
    --cc=cpebenito@tresys.com \
    --cc=dwalsh@redhat.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.