summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNIIBE Yutaka <gniibe@fsij.org>2022-06-22 06:34:06 +0200
committerNIIBE Yutaka <gniibe@fsij.org>2022-06-22 06:34:06 +0200
commitfe535cf26592d6af53d5aea15b3fdc1d79ba0f80 (patch)
tree2294d958f5403bb0317e3f60c89b74a3902a3012
parentsm: Improve pkcs#12 debug output. (diff)
downloadgnupg2-fe535cf26592d6af53d5aea15b3fdc1d79ba0f80.tar.xz
gnupg2-fe535cf26592d6af53d5aea15b3fdc1d79ba0f80.zip
agent,gpg,tools: Fix use of log_get_fd.
* agent/call-daemon.c (daemon_start): Don't put file descriptor from log_get_fd to no_close_list. * agent/call-pinentry.c (start_pinentry): Likewise. * common/call-gpg.c (start_gpg): Likewise. * call-syshelp.c (start_syshelp): Likewise. * tools/gpg-connect-agent.c (main): Likewise. -- GnuPG-bug-id: 5921 Signed-off-by: NIIBE Yutaka <gniibe@fsij.org>
-rw-r--r--agent/call-daemon.c6
-rw-r--r--agent/call-pinentry.c6
-rw-r--r--common/call-gpg.c2
-rw-r--r--g13/call-syshelp.c7
-rw-r--r--tools/gpg-connect-agent.c3
5 files changed, 5 insertions, 19 deletions
diff --git a/agent/call-daemon.c b/agent/call-daemon.c
index f907f40eb..0c3605274 100644
--- a/agent/call-daemon.c
+++ b/agent/call-daemon.c
@@ -414,11 +414,7 @@ daemon_start (enum daemon_type type, ctrl_t ctrl)
i=0;
if (!opt.running_detached)
- {
- if (log_get_fd () != -1)
- no_close_list[i++] = assuan_fd_from_posix_fd (log_get_fd ());
- no_close_list[i++] = assuan_fd_from_posix_fd (fileno (stderr));
- }
+ no_close_list[i++] = assuan_fd_from_posix_fd (fileno (stderr));
no_close_list[i] = ASSUAN_INVALID_FD;
/* Connect to the daemon and perform initial handshaking. Use
diff --git a/agent/call-pinentry.c b/agent/call-pinentry.c
index bb55a3714..711758efc 100644
--- a/agent/call-pinentry.c
+++ b/agent/call-pinentry.c
@@ -394,11 +394,7 @@ start_pinentry (ctrl_t ctrl)
i=0;
if (!opt.running_detached)
- {
- if (log_get_fd () != -1)
- no_close_list[i++] = assuan_fd_from_posix_fd (log_get_fd ());
- no_close_list[i++] = assuan_fd_from_posix_fd (fileno (stderr));
- }
+ no_close_list[i++] = assuan_fd_from_posix_fd (fileno (stderr));
no_close_list[i] = ASSUAN_INVALID_FD;
rc = assuan_new (&ctx);
diff --git a/common/call-gpg.c b/common/call-gpg.c
index c1472e961..a4723ca43 100644
--- a/common/call-gpg.c
+++ b/common/call-gpg.c
@@ -106,8 +106,6 @@ start_gpg (ctrl_t ctrl, const char *gpg_program, strlist_t gpg_arguments,
argv[i++] = NULL;
i = 0;
- if (log_get_fd () != -1)
- no_close_list[i++] = assuan_fd_from_posix_fd (log_get_fd ());
no_close_list[i++] = assuan_fd_from_posix_fd (fileno (stderr));
if (input_fd != -1)
no_close_list[i++] = assuan_fd_from_posix_fd (input_fd);
diff --git a/g13/call-syshelp.c b/g13/call-syshelp.c
index a69573bd1..8850c3779 100644
--- a/g13/call-syshelp.c
+++ b/g13/call-syshelp.c
@@ -102,11 +102,8 @@ start_syshelp (ctrl_t ctrl, assuan_context_t *r_ctx)
return err;
}
- i = 0;
- if (log_get_fd () != -1)
- no_close_list[i++] = assuan_fd_from_posix_fd (log_get_fd ());
- no_close_list[i++] = assuan_fd_from_posix_fd (es_fileno (es_stderr));
- no_close_list[i] = ASSUAN_INVALID_FD;
+ no_close_list[0] = assuan_fd_from_posix_fd (es_fileno (es_stderr));
+ no_close_list[1] = ASSUAN_INVALID_FD;
err = assuan_new (&ctx);
if (err)
diff --git a/tools/gpg-connect-agent.c b/tools/gpg-connect-agent.c
index dff367050..7a0dc65b9 100644
--- a/tools/gpg-connect-agent.c
+++ b/tools/gpg-connect-agent.c
@@ -1319,8 +1319,7 @@ main (int argc, char **argv)
assuan_fd_t no_close[3];
no_close[0] = assuan_fd_from_posix_fd (es_fileno (es_stderr));
- no_close[1] = assuan_fd_from_posix_fd (log_get_fd ());
- no_close[2] = ASSUAN_INVALID_FD;
+ no_close[1] = ASSUAN_INVALID_FD;
rc = assuan_new (&ctx);
if (rc)