From: Sven Vermeulen <sven.vermeulen@siphos.be>
To: selinux@tycho.nsa.gov
Subject: sepolgen requires unofficial setools patch
Date: Mon, 21 May 2012 22:58:50 +0200 [thread overview]
Message-ID: <20120521205849.GA8511@siphos.be> (raw)
Hi guys,
It looks like the current stable sepolgen release has requirements towards
an unofficial (well, fedora/rhel only) patch on setools. With the current
stable setools, it gives the following error when trying to use audit2allow
on a denial that contains write & open:
Traceback (most recent call last):
File "/usr/bin/audit2allow-2.7", line 354, in <module>
app.main()
File "/usr/bin/audit2allow-2.7", line 345, in main
self.__output()
File "/usr/bin/audit2allow-2.7", line 315, in __output
g.add_access(self.__avs)
File "/usr/lib64/python2.7/site-packages/sepolgen/policygen.py", line 211, in add_access
self.__add_allow_rules(raw_allow)
File "/usr/lib64/python2.7/site-packages/sepolgen/policygen.py", line 179, in __add_allow_rules
self.domains = seinfo(ATTRIBUTE, name="domain")[0]["types"]
NameError: global name 'seinfo' is not defined
The patch that RedHat (and Fedora) provides fixes this in Python 2 systems,
but doesn't work in Python 3 (because Python 3 has a different setup for
Extension-based modules). I have a locally-tested patch on that, but I'm not
sure this is a good way to go forward.
Perhaps it would be wise to remove the dependency towards the setools
binding and instead include the necessary code in the userspace libraries
themselves? policygen.py doesn't require the entire set of querying that
seinfo provides...
The patch that is suggested by RedHat/Fedora doesn't follow the same
structure as the other bindings do (like libqpol/libapol) in setools too.
Wkr,
Sven Vermeulen
--
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.
next reply other threads:[~2012-05-21 21:00 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-21 20:58 Sven Vermeulen [this message]
2012-05-23 15:46 ` sepolgen requires unofficial setools patch Daniel J Walsh
2012-05-23 17:32 ` Christopher J. PeBenito
2012-05-23 17:46 ` Daniel J Walsh
2012-05-23 18:29 ` Christopher J. PeBenito
2012-05-23 18:46 ` Daniel J Walsh
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=20120521205849.GA8511@siphos.be \
--to=sven.vermeulen@siphos.be \
--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.