[libvirt PATCH 2/2] util: Remove virCommandPassFDIndex()

Jonathon Jongsma jjongsma at redhat.com
Fri May 13 19:32:42 UTC 2022


The last use of this function was removed in commit 6d161bcc, so the
function is no longer used except as an internal implementation for
virCommandPassFD().

Signed-off-by: Jonathon Jongsma <jjongsma at redhat.com>
---
 src/libvirt_private.syms |  1 -
 src/util/vircommand.c    | 28 ++--------------------------
 2 files changed, 2 insertions(+), 27 deletions(-)

diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 66f73e84aa..d95c181793 100644
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -2057,7 +2057,6 @@ virCommandNewArgs;
 virCommandNewVAList;
 virCommandNonblockingFDs;
 virCommandPassFD;
-virCommandPassFDIndex;
 virCommandRawStatus;
 virCommandRequireHandshake;
 virCommandRun;
diff --git a/src/util/vircommand.c b/src/util/vircommand.c
index f3c26a2673..d78c666f28 100644
--- a/src/util/vircommand.c
+++ b/src/util/vircommand.c
@@ -1021,11 +1021,10 @@ virCommandNewVAList(const char *binary, va_list list)
         VIR_FORCE_CLOSE(fd)
 
 /**
- * virCommandPassFDIndex:
+ * virCommandPassFD:
  * @cmd: the command to modify
  * @fd: fd to reassign to the child
  * @flags: extra flags; binary-OR of virCommandPassFDFlags
- * @idx: pointer to fill with the index of the FD in the transfer set
  *
  * Transfer the specified file descriptor to the child, instead
  * of closing it on exec. @fd must not be one of the three
@@ -1036,7 +1035,7 @@ virCommandNewVAList(const char *binary, va_list list)
  * should cease using the @fd when this call completes
  */
 void
-virCommandPassFDIndex(virCommand *cmd, int fd, unsigned int flags, size_t *idx)
+virCommandPassFD(virCommand *cmd, int fd, unsigned int flags)
 {
     if (!cmd) {
         VIR_COMMAND_MAYBE_CLOSE_FD(fd, flags);
@@ -1052,29 +1051,6 @@ virCommandPassFDIndex(virCommand *cmd, int fd, unsigned int flags, size_t *idx)
     }
 
     virCommandFDSet(cmd, fd, flags);
-
-    if (idx)
-        *idx = cmd->npassfd - 1;
-}
-
-/**
- * virCommandPassFD:
- * @cmd: the command to modify
- * @fd: fd to reassign to the child
- * @flags: extra flags; binary-OR of virCommandPassFDFlags
- *
- * Transfer the specified file descriptor to the child, instead
- * of closing it on exec. @fd must not be one of the three
- * standard streams.
- *
- * If the flag VIR_COMMAND_PASS_FD_CLOSE_PARENT is set then fd will
- * be closed in the parent no later than Run/RunAsync/Free. The parent
- * should cease using the @fd when this call completes
- */
-void
-virCommandPassFD(virCommand *cmd, int fd, unsigned int flags)
-{
-    virCommandPassFDIndex(cmd, fd, flags, NULL);
 }
 
 /**
-- 
2.35.3



More information about the libvir-list mailing list