File 0001-util-Don-t-spawn-pkttyagent-when-stdin-is-... of Package libvirt

Overview Repositories Revisions Requests Users Attributes Meta

File 0001-util-Don-t-spawn-pkttyagent-when-stdin-is-not-a-tty.patch of Package libvirt (Revision 8ba47c16c283e1b1117709710069d07a)

Currently displaying revision 8ba47c16c283e1b1117709710069d07a , Show latest

From d151208395e065c340bee14e77b091c840e55aeb Mon Sep 17 00:00:00 2001
From: Jim Fehlig <jfehlig@suse.com>
Date: Mon, 27 Jun 2022 15:41:02 -0600
Subject: util: Don't spawn pkttyagent when stdin is not a tty

My idea was that running pkttyagent unconditionally, modulo checks that
pkttyagent itself does to make sure it does not fail, is not going to be an
issue turned out to be wrong.  Adding back the original check for stdin being a
tty helps in some testing scenarios as reported by Jim Fehlig and does not
really cause any issues.  I originally wanted it in because it also made
pkttyagent auth work with redirected input into virsh (with a connection that
requires polkit authentication and without a session-wide polkit tty agent,
basically making pkttyagent necessary to succeed).  But anyone running virsh
like that is asking for problems already anyway =)

Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
---
 src/util/virpolkit.c | 3 +++
 1 file changed, 3 insertions(+)

Index: libvirt-8.0.0/src/util/virpolkit.c
===================================================================
--- libvirt-8.0.0.orig/src/util/virpolkit.c
+++ libvirt-8.0.0/src/util/virpolkit.c
@@ -237,6 +237,9 @@ virPolkitAgentAvailable(void)
     const char *termid = ctermid(NULL);
     VIR_AUTOCLOSE fd = -1;
 
+    if (!isatty(STDIN_FILENO))
+        return false;
+
     if (!virFileIsExecutable(PKTTYAGENT))
         return false;

Places

File 0001-util-Don-t-spawn-pkttyagent-when-stdin-is-not-a-tty.patch of Package libvirt (Revision 8ba47c16c283e1b1117709710069d07a)

Places