Re: libsepol policycap names

2018-03-07 Thread Jason Zaman
On Wed, Mar 07, 2018 at 04:19:33PM +0800, Jason Zaman wrote:
> On Mon, Mar 05, 2018 at 09:03:10AM -0500, Stephen Smalley wrote:
> > On 03/02/2018 01:49 PM, Chris PeBenito wrote:
> > > I've been able to make SETools dynamically link to libsepol.  However,
> > > one challenge is with policycap names.  They're static libsepol, with
> > > nothing that exports them.  Can we either:
> > > 
> > > * export the sepol_polcap_getname() function, or
> > > * move the polcap_names[] in polcaps.c into  ?
> > > 
> > > Then I can avoid having to manually keep a polcap name list inside 
> > > SETools.
> > 
> > Let's do the former (and also export sepol_polcap_getnum for
> > consistency).  Did you want to spin up a patch or were you asking us to
> > do so?
> 
> If we put polcap_names[] in the .h file, would it mean that setools and
> any other programs need to be re-compiled to see any changes? if thats
> the case then only sepol_polcap_getname() would be easier for
> maintenance.

Oops i read it wrong, you mean export both the functions not the array,
in that case yes agreed completely.

-- Jason




Re: libsepol policycap names

2018-03-07 Thread Jason Zaman
On Mon, Mar 05, 2018 at 09:03:10AM -0500, Stephen Smalley wrote:
> On 03/02/2018 01:49 PM, Chris PeBenito wrote:
> > I've been able to make SETools dynamically link to libsepol.  However,
> > one challenge is with policycap names.  They're static libsepol, with
> > nothing that exports them.  Can we either:
> > 
> > * export the sepol_polcap_getname() function, or
> > * move the polcap_names[] in polcaps.c into  ?
> > 
> > Then I can avoid having to manually keep a polcap name list inside SETools.
> 
> Let's do the former (and also export sepol_polcap_getnum for
> consistency).  Did you want to spin up a patch or were you asking us to
> do so?

If we put polcap_names[] in the .h file, would it mean that setools and
any other programs need to be re-compiled to see any changes? if thats
the case then only sepol_polcap_getname() would be easier for
maintenance.

-- Jason





Re: libsepol policycap names

2018-03-05 Thread Stephen Smalley
On 03/02/2018 01:49 PM, Chris PeBenito wrote:
> I've been able to make SETools dynamically link to libsepol.  However,
> one challenge is with policycap names.  They're static libsepol, with
> nothing that exports them.  Can we either:
> 
> * export the sepol_polcap_getname() function, or
> * move the polcap_names[] in polcaps.c into  ?
> 
> Then I can avoid having to manually keep a polcap name list inside SETools.

Let's do the former (and also export sepol_polcap_getnum for
consistency).  Did you want to spin up a patch or were you asking us to
do so?






Re: libsepol policycap names

2018-03-03 Thread William Roberts
On Fri, Mar 2, 2018 at 10:49 AM, Chris PeBenito  wrote:
> I've been able to make SETools dynamically link to libsepol.  However, one
> challenge is with policycap names.  They're static libsepol, with nothing
> that exports them.  Can we either:
>
> * export the sepol_polcap_getname() function, or
> * move the polcap_names[] in polcaps.c into  ?
>
> Then I can avoid having to manually keep a polcap name list inside SETools.

I would rather export the sepol_polcap_getname() rather than the raw mapping
array.

>
> --
> Chris PeBenito
>



libsepol policycap names

2018-03-02 Thread Chris PeBenito
I've been able to make SETools dynamically link to libsepol.  However, 
one challenge is with policycap names.  They're static libsepol, with 
nothing that exports them.  Can we either:


* export the sepol_polcap_getname() function, or
* move the polcap_names[] in polcaps.c into  ?

Then I can avoid having to manually keep a polcap name list inside SETools.

--
Chris PeBenito