On Tue, 2016-01-19 at 10:34 -0500, Simo Sorce wrote:
On Tue, 2016-01-19 at 11:23 +0100, Lukas Slebodnik wrote:
[...]
> >+#endif /* __SSSD_UTIL_SELINUX_H__ */
> BTW will we need this header file if we make
> struct cli_creds opaque?
Replying to both your mails here:
Making cli_creds opaque requires using a pointer and dealing with
allocating it, I guess I can do that, the headers file would still be
needed in order to avoid huge ifdefs around the functions that implement
handling SELinux stuff. It makes the code a lot more readable and
searchable.
Simo.
Attached find patch that changes the code so that cli_creds is now
opaque.
This *should* work w/o the patch that changes the headers but I haven't
tested it w/o that patch yet.
Simo.
--
Simo Sorce * Red Hat, Inc * New York