On Tue, Mar 26, 2019 at 04:24:00PM +0000, Daniel P. Berrangé wrote:
On Tue, Mar 26, 2019 at 05:06:16PM +0100, Andrea Bolognani wrote:
> Running QEMU as root is a pretty bad idea, so try to make the
> user aware of that as part of the configure summary.
>
> Signed-off-by: Andrea Bolognani <abologna(a)redhat.com>
> ---
> m4/virt-driver-qemu.m4 | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/m4/virt-driver-qemu.m4 b/m4/virt-driver-qemu.m4
> index 5d4ac66a4b..a8b9fe54b3 100644
> --- a/m4/virt-driver-qemu.m4
> +++ b/m4/virt-driver-qemu.m4
> @@ -104,5 +104,10 @@ AC_DEFUN([LIBVIRT_DRIVER_RESULT_QEMU], [
> ])
>
> AC_DEFUN([LIBVIRT_RESULT_QEMU_PRIVILEGES], [
> - LIBVIRT_RESULT([QEMU], [$QEMU_USER:$QEMU_GROUP])
> + if test "$QEMU_USER" = "root"; then
> + LIBVIRT_RESULT([QEMU], [$QEMU_USER:$QEMU_GROUP],
> + [!!! running QEMU as root is strongly discouraged !!!])
Am I evil for suggesting use of blink :-P
Yes.
Jano