All of lore.kernel.org
 help / color / mirror / Atom feed
From: cpebenito@tresys.com (Christopher J. PeBenito)
To: refpolicy@oss.tresys.com
Subject: [refpolicy] modules_object_t vs. modules_dep_t labeling
Date: Mon, 5 Oct 2015 15:17:48 -0400	[thread overview]
Message-ID: <5612CCDC.3020900@tresys.com> (raw)
In-Reply-To: <20151005163442.GB21879@x250>

On 10/5/2015 12:34 PM, Dominick Grift wrote:
> On Thu, Oct 01, 2015 at 11:58:25AM +0200, Miroslav Grepl wrote:
>> We have more and more bugs with mislabeled /lib/modules/*/modules.dep*
>> files. There is a default label for them - modules_dep_t but we get them
>> labeled as modules_object_t. Yes, we can add filename transition rules
>> and also find a reason why they get wrong labeling (in progress).
> 
>> But is there a big advantage to have these two labels. At least I don't
>> see it from the policy point of view (sesearch).
> 
>> Thank you.
> 
> 
> Still not verified but:
> 
> /sbin/depmod is a link to /bin/kmod
> 
> So i suspect /bin/kmod now creates the modules_dep files via rpm_script_t %post and
> the /sbin/new_kernel_pkg shell script:
> 
> doDepmod() {
>     [ -n "$verbose" ] && echo "running depmod for $version"
>         depmod -ae -F /boot/System.map-$version $version
>         }
> 
> but because insmod_t is lacking the appropriate auto object type transitions and because insmod is
> unconfined, the files get created with the wrong label.
> 
> So you should copy the auto object type transition rules for modules_dep
> from depmod to insmod i suspect
> 
> I would not want insmod_t to be able to mess with module_object_t type
> files.
> 
> But yes, in Fedora insmod is unconfined...

Thanks for digging through this issue.  For the time being, we'll keep
what we have.  Miroslav, if the type transition Dominick suggests works,
then we can put it in refpolicy.


-- 
Chris PeBenito
Tresys Technology, LLC
www.tresys.com | oss.tresys.com

  reply	other threads:[~2015-10-05 19:17 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-01  9:58 [refpolicy] modules_object_t vs. modules_dep_t labeling Miroslav Grepl
2015-10-05 12:14 ` Christopher J. PeBenito
2015-10-05 12:48 ` Dominick Grift
2015-10-06 18:13   ` Dominick Grift
2015-10-05 16:34 ` Dominick Grift
2015-10-05 19:17   ` Christopher J. PeBenito [this message]
2015-10-06 11:29     ` Dominick Grift
2015-10-06 11:46       ` Dominick Grift
2015-10-08  5:13         ` Miroslav Grepl
2015-10-08 13:15           ` Christopher J. PeBenito
2015-10-09  7:17             ` Miroslav Grepl
2015-10-10  7:17             ` Sven Vermeulen
2015-10-10 12:46               ` Daniel J Walsh
2015-10-10 13:40                 ` Dominick Grift
2015-10-12  7:23                   ` Miroslav Grepl

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=5612CCDC.3020900@tresys.com \
    --to=cpebenito@tresys.com \
    --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.