Re: [PATCH] Revised Patch for local nodecon support in semanage (was: Adding local nodecon's through semanage)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2008-08-21 at 16:59 -0400, Stephen Smalley wrote:
> On Thu, 2008-08-14 at 09:32 +0200, Christian Kuester wrote:
> > Stephen Smalley schrieb:
> > 
> > Hi List,
> > 
> > > On Tue, 2008-07-08 at 08:30 -0400, Stephen Smalley wrote:
> > >> On Tue, 2008-07-08 at 12:13 +0200, Christian Kuester wrote:
> > >>>> Other tidbits on the semanage patch that I noticed:
> > >>>> - semanage node -l was broken, requires additional argument that has
> > >>>> been added to the list methods subsequently.  Also would be nice to
> > >>>> support locallist/-C option.
> > >>>> - semanage node -p option should take a string rather than an integer
> > >>>> and map it to the proper symbolic constant for ipv4/ipv6.
> > >> Please be sure to test each of the nodeRecords methods.
> > > Are you still pursuing getting this cleaned up and merged?
> > 
> > Sorry, it took some time. The revised patch for nodecon support in
> > the semanage tool is attached.
> > 
> > It now takes strings as arguments for the ip protocol. list/locallist
> > work as expected and output is more readable. I also made changes for
> > the semanage.8 man page.
> 
> semanage node -lC appears to list all of the entries rather than only
> the local modifications (i.e. the ones in the nodes.local file).
> Compare with semanage fcontext -lC or port -lC.
> 

Ok, this turned out to be trivial - just needed to pass the localist
value to the get_all() call. Fixed and committed, thanks.

-- 
Stephen Smalley
National Security Agency


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with
the words "unsubscribe selinux" without quotes as the message.

[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux