diff --git a/configure.ac b/configure.ac index 05a36cbe42..2cff562106 100644 --- a/configure.ac +++ b/configure.ac @@ -1113,9 +1113,9 @@ if test "x$with_polkit" = "xyes" || test "x$with_polkit" = "xcheck"; then AC_PATH_PROG([PKCHECK_PATH],[pkcheck], [], [/usr/sbin:$PATH]) if test "x$PKCHECK_PATH" != "x" ; then AC_DEFINE_UNQUOTED([PKCHECK_PATH],["$PKCHECK_PATH"],[Location of pkcheck program]) - AC_DEFINE_UNQUOTED([HAVE_POLKIT], 1, + AC_DEFINE_UNQUOTED([WITH_POLKIT], 1, [use PolicyKit for UNIX socket access checks]) - AC_DEFINE_UNQUOTED([HAVE_POLKIT1], 1, + AC_DEFINE_UNQUOTED([WITH_POLKIT1], 1, [use PolicyKit for UNIX socket access checks]) with_polkit="yes" with_polkit1="yes" @@ -1131,9 +1131,9 @@ if test "x$with_polkit" = "xyes" || test "x$with_polkit" = "xcheck"; then fi ]) if test "x$with_polkit" = "xyes" ; then - AC_DEFINE_UNQUOTED([HAVE_POLKIT], 1, + AC_DEFINE_UNQUOTED([WITH_POLKIT], 1, [use PolicyKit for UNIX socket access checks]) - AC_DEFINE_UNQUOTED([HAVE_POLKIT0], 1, + AC_DEFINE_UNQUOTED([WITH_POLKIT0], 1, [use PolicyKit for UNIX socket access checks]) old_CFLAGS=$CFLAGS @@ -1152,9 +1152,9 @@ if test "x$with_polkit" = "xyes" || test "x$with_polkit" = "xcheck"; then fi fi fi -AM_CONDITIONAL([HAVE_POLKIT], [test "x$with_polkit" = "xyes"]) -AM_CONDITIONAL([HAVE_POLKIT0], [test "x$with_polkit0" = "xyes"]) -AM_CONDITIONAL([HAVE_POLKIT1], [test "x$with_polkit1" = "xyes"]) +AM_CONDITIONAL([WITH_POLKIT], [test "x$with_polkit" = "xyes"]) +AM_CONDITIONAL([WITH_POLKIT0], [test "x$with_polkit0" = "xyes"]) +AM_CONDITIONAL([WITH_POLKIT1], [test "x$with_polkit1" = "xyes"]) AC_SUBST([POLKIT_CFLAGS]) AC_SUBST([POLKIT_LIBS]) diff --git a/daemon/Makefile.am b/daemon/Makefile.am index c4faa11783..f2572724ec 100644 --- a/daemon/Makefile.am +++ b/daemon/Makefile.am @@ -170,8 +170,8 @@ endif libvirtd_LDADD += ../src/libvirt.la -if HAVE_POLKIT -if HAVE_POLKIT0 +if WITH_POLKIT +if WITH_POLKIT0 policydir = $(datadir)/PolicyKit/policy policyauth = auth_admin_keep_session else @@ -201,7 +201,7 @@ uninstall-local:: uninstall-init-redhat uninstall-init-systemd uninstall-init-up rmdir $(DESTDIR)$(localstatedir)/run/libvirt || : rmdir $(DESTDIR)$(localstatedir)/lib/libvirt || : -if HAVE_POLKIT +if WITH_POLKIT install-data-polkit:: $(MKDIR_P) $(DESTDIR)$(policydir) $(INSTALL_DATA) libvirtd.policy $(DESTDIR)$(policydir)/org.libvirt.unix.policy diff --git a/daemon/libvirtd-config.c b/daemon/libvirtd-config.c index fe9fc10a59..5e3ae218f3 100644 --- a/daemon/libvirtd-config.c +++ b/daemon/libvirtd-config.c @@ -242,7 +242,7 @@ daemonConfigNew(bool privileged ATTRIBUTE_UNUSED) goto no_memory; /* Only default to PolicyKit if running as root */ -#if HAVE_POLKIT +#if WITH_POLKIT if (privileged) { data->auth_unix_rw = REMOTE_AUTH_POLKIT; data->auth_unix_ro = REMOTE_AUTH_POLKIT; @@ -250,7 +250,7 @@ daemonConfigNew(bool privileged ATTRIBUTE_UNUSED) #endif data->auth_unix_rw = REMOTE_AUTH_NONE; data->auth_unix_ro = REMOTE_AUTH_NONE; -#if HAVE_POLKIT +#if WITH_POLKIT } #endif @@ -374,7 +374,7 @@ daemonConfigLoadOptions(struct daemonConfig *data, if (remoteConfigGetAuth(conf, "auth_unix_rw", &data->auth_unix_rw, filename) < 0) goto error; -#if HAVE_POLKIT +#if WITH_POLKIT /* Change default perms to be wide-open if PolicyKit is enabled. * Admin can always override in config file */ diff --git a/daemon/remote.c b/daemon/remote.c index 7b1f8ebe62..6cdfcd87db 100644 --- a/daemon/remote.c +++ b/daemon/remote.c @@ -24,7 +24,7 @@ #include "virerror.h" -#if HAVE_POLKIT0 +#if WITH_POLKIT0 # include # include #endif @@ -2802,7 +2802,7 @@ remoteDispatchAuthSaslStep(virNetServerPtr server ATTRIBUTE_UNUSED, -#if HAVE_POLKIT1 +#if WITH_POLKIT1 static int remoteDispatchAuthPolkit(virNetServerPtr server ATTRIBUTE_UNUSED, virNetServerClientPtr client, @@ -2912,7 +2912,7 @@ authdeny: client, REMOTE_AUTH_POLKIT, ident); goto error; } -#elif HAVE_POLKIT0 +#elif WITH_POLKIT0 static int remoteDispatchAuthPolkit(virNetServerPtr server ATTRIBUTE_UNUSED, virNetServerClientPtr client, @@ -3056,7 +3056,7 @@ authdeny: goto error; } -#else /* !HAVE_POLKIT0 & !HAVE_POLKIT1*/ +#else /* !WITH_POLKIT0 & !HAVE_POLKIT1*/ static int remoteDispatchAuthPolkit(virNetServerPtr server ATTRIBUTE_UNUSED, @@ -3071,7 +3071,7 @@ remoteDispatchAuthPolkit(virNetServerPtr server ATTRIBUTE_UNUSED, virNetMessageSaveError(rerr); return -1; } -#endif /* HAVE_POLKIT1 */ +#endif /* WITH_POLKIT1 */ /*************************************************************** diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c index dcb06c9726..916ded5916 100644 --- a/src/remote/remote_driver.c +++ b/src/remote/remote_driver.c @@ -122,10 +122,10 @@ static int remoteAuthenticate(virConnectPtr conn, struct private_data *priv, static int remoteAuthSASL(virConnectPtr conn, struct private_data *priv, virConnectAuthPtr auth, const char *mech); #endif -#if HAVE_POLKIT +#if WITH_POLKIT static int remoteAuthPolkit(virConnectPtr conn, struct private_data *priv, virConnectAuthPtr auth); -#endif /* HAVE_POLKIT */ +#endif /* WITH_POLKIT */ static virDomainPtr get_nonnull_domain(virConnectPtr conn, remote_nonnull_domain domain); static virNetworkPtr get_nonnull_network(virConnectPtr conn, remote_nonnull_network network); @@ -3504,7 +3504,7 @@ remoteAuthenticate(virConnectPtr conn, struct private_data *priv, } #endif -#if HAVE_POLKIT +#if WITH_POLKIT case REMOTE_AUTH_POLKIT: if (remoteAuthPolkit(conn, priv, auth) < 0) { VIR_FREE(ret.types.types_val); @@ -4075,8 +4075,8 @@ remoteAuthSASL(virConnectPtr conn, struct private_data *priv, #endif /* WITH_SASL */ -#if HAVE_POLKIT -# if HAVE_POLKIT1 +#if WITH_POLKIT +# if WITH_POLKIT1 static int remoteAuthPolkit(virConnectPtr conn, struct private_data *priv, virConnectAuthPtr auth ATTRIBUTE_UNUSED) @@ -4094,7 +4094,7 @@ remoteAuthPolkit(virConnectPtr conn, struct private_data *priv, VIR_DEBUG("PolicyKit-1 authentication complete"); return 0; } -# elif HAVE_POLKIT0 +# elif WITH_POLKIT0 /* Perform the PolicyKit authentication process */ static int @@ -4156,8 +4156,8 @@ out: VIR_DEBUG("PolicyKit-0 authentication complete"); return 0; } -# endif /* HAVE_POLKIT0 */ -#endif /* HAVE_POLKIT */ +# endif /* WITH_POLKIT0 */ +#endif /* WITH_POLKIT */ /*----------------------------------------------------------------------*/ static int remoteDomainEventRegister(virConnectPtr conn,