All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel J Walsh <dwalsh@redhat.com>
To: SE Linux <selinux@tycho.nsa.gov>
Subject: Is it time we removed the version number from policy.22/policy21?
Date: Fri, 25 Jan 2008 11:29:47 -0500	[thread overview]
Message-ID: <479A0E7B.7000109@redhat.com> (raw)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

I just got burned by the policy version bumping.  I blogged about how
cool audit2why is and then policycoreutils/libselinux had a bug.

They were hard coded to use the policy version of the kernel that was
running to look for policy.

They were both doing the equivalent of
/etc/selinux/targeted/policy/policy.`cat /selinux/policyvers`

But I guess we just bumped the version of policy to 22 in libsepol.  (My
mistake for not noticing).  But the kernel still only supports 21.

So a freshly installed machine has 22 on it and audit2why blows up with
a missing policy because it is looking for 21.  On my test machine I
have a policy.21 and a policy.22 so I never noticed.  (this in my
opinion is a bug in semanage.  It should have cleaned up the old version.

I think this just points out the problem of adding the version number to
the policy file on disk.  This really serves no purpose other than to
create bugs every time we bump the version.

I would like to suggest that we switch to just building
/etc/selinux/TYPE/policy/policy
and have a symbolic link for backwards compatibility
/etc/selinux/TYPE/policy/policy.22 -> policy
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.8 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org

iEYEARECAAYFAkeaDnoACgkQrlYvE4MpobPKgACfXe34vNKs7cObfgncFZUoJTte
1DMAoN58Hkba5GzZzGhO0vddOiMQu+g5
=XU4s
-----END PGP SIGNATURE-----

--
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:[~2008-01-25 16:29 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-25 16:29 Daniel J Walsh [this message]
2008-01-25 16:42 ` Is it time we removed the version number from policy.22/policy21? Stephen Smalley
2008-01-25 16:44   ` Stephen Smalley

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=479A0E7B.7000109@redhat.com \
    --to=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.