On Thu, Jul 09, 2020 at 05:00:39PM +0200, Ján Tomko wrote:
On a Thursday in 2020, Pavel Hrdina wrote:
> All of the listed functions are available in libselinux version 2.2.
> Our supported OSes start with version 2.5 so there is no need to check
> it.
>
> Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com>
> ---
> m4/virt-secdriver-selinux.m4 | 24 ++----------------------
> 1 file changed, 2 insertions(+), 22 deletions(-)
This doesn't look right with no changes to the SELinux driver.
>
> diff --git a/m4/virt-secdriver-selinux.m4 b/m4/virt-secdriver-selinux.m4
> index a48569fc33a..4174249a510 100644
> --- a/m4/virt-secdriver-selinux.m4
> +++ b/m4/virt-secdriver-selinux.m4
> @@ -32,28 +32,8 @@ AC_DEFUN([LIBVIRT_SECDRIVER_CHECK_SELINUX], [
> AC_MSG_ERROR([You must install the libselinux development package and enable
SELinux with the --with-selinux=yes in order to compile libvirt
--with-secdriver-selinux=yes])
> fi
> elif test "$with_secdriver_selinux" != "no"; then
> - old_CFLAGS="$CFLAGS"
> - old_LIBS="$LIBS"
> - CFLAGS="$CFLAGS $SELINUX_CFLAGS"
> - LIBS="$CFLAGS $SELINUX_LIBS"
> -
> - fail=0
> - AC_CHECK_FUNC([selinux_virtual_domain_context_path], [], [fail=1])
> - AC_CHECK_FUNC([selinux_virtual_image_context_path], [], [fail=1])
> - AC_CHECK_FUNCS([selinux_lxc_contexts_path])
This means the code relying on HAVE_SELINUX_LXC_CONTEXTS_PATH will no
longer be compiled.
Jano
Nice catch, thanks, I'll fix that in v2.
Pavel