diff options
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/gpg-connect-agent.c | 28 | ||||
| -rw-r--r-- | tools/gpgconf-comp.c | 55 |
2 files changed, 66 insertions, 17 deletions
diff --git a/tools/gpg-connect-agent.c b/tools/gpg-connect-agent.c index 78dea2a4b..8e8ebcb71 100644 --- a/tools/gpg-connect-agent.c +++ b/tools/gpg-connect-agent.c @@ -63,7 +63,8 @@ enum cmd_and_opt_values oHex, oDecode, oNoExtConnect, - oDirmngr + oDirmngr, + oNoAutostart, }; @@ -89,6 +90,7 @@ static ARGPARSE_OPTS opts[] = { N_("|FILE|run commands from FILE on startup")), ARGPARSE_s_n (oSubst, "subst", N_("run /subst on startup")), + ARGPARSE_s_n (oNoAutostart, "no-autostart", "@"), ARGPARSE_s_n (oNoVerbose, "no-verbose", "@"), ARGPARSE_s_s (oHomedir, "homedir", "@" ), ARGPARSE_s_s (oAgentProgram, "agent-program", "@"), @@ -103,6 +105,7 @@ struct { int verbose; /* Verbosity level. */ int quiet; /* Be extra quiet. */ + int autostart; /* Start the server if not running. */ const char *homedir; /* Configuration directory name */ const char *agent_program; /* Value of --agent-program. */ const char *dirmngr_program; /* Value of --dirmngr-program. */ @@ -1175,6 +1178,7 @@ main (int argc, char **argv) opt.homedir = default_homedir (); + opt.autostart = 1; opt.connect_flags = 1; /* Parse the command line. */ @@ -1191,6 +1195,7 @@ main (int argc, char **argv) case oHomedir: opt.homedir = pargs.r.ret_str; break; case oAgentProgram: opt.agent_program = pargs.r.ret_str; break; case oDirmngrProgram: opt.dirmngr_program = pargs.r.ret_str; break; + case oNoAutostart: opt.autostart = 0; break; case oHex: opt.hex = 1; break; case oDecode: opt.decode = 1; break; case oDirmngr: opt.use_dirmngr = 1; break; @@ -2195,6 +2200,7 @@ start_agent (void) GPG_ERR_SOURCE_DEFAULT, opt.homedir, opt.dirmngr_program, + opt.autostart, !opt.quiet, 0, NULL, NULL); else @@ -2204,14 +2210,30 @@ start_agent (void) opt.agent_program, NULL, NULL, session_env, + opt.autostart, !opt.quiet, 0, NULL, NULL); session_env_release (session_env); if (err) { - log_error (_("error sending standard options: %s\n"), gpg_strerror (err)); - exit (1); + if (!opt.autostart + && (gpg_err_code (err) + == opt.use_dirmngr? GPG_ERR_NO_DIRMNGR : GPG_ERR_NO_AGENT)) + { + /* In the no-autostart case we don't make gpg-connect-agent + fail on a missing server. */ + log_info (opt.use_dirmngr? + _("no dirmngr running in this session\n"): + _("no gpg-agent running in this session\n")); + exit (0); + } + else + { + log_error (_("error sending standard options: %s\n"), + gpg_strerror (err)); + exit (1); + } } return ctx; diff --git a/tools/gpgconf-comp.c b/tools/gpgconf-comp.c index f9999bac6..77139bbae 100644 --- a/tools/gpgconf-comp.c +++ b/tools/gpgconf-comp.c @@ -107,6 +107,7 @@ gc_error (int status, int errnum, const char *fmt, ...) /* Forward declaration. */ static void gpg_agent_runtime_change (int killflag); static void scdaemon_runtime_change (int killflag); +static void dirmngr_runtime_change (int killflag); /* Backend configuration. Backends are used to decide how the default and current value of an option can be determined, and how the @@ -189,7 +190,7 @@ static struct { SCDAEMON_DISP_NAME, SCDAEMON_NAME, GNUPG_MODULE_NAME_SCDAEMON, scdaemon_runtime_change, GPGCONF_NAME"-" SCDAEMON_NAME ".conf" }, { DIRMNGR_DISP_NAME, DIRMNGR_NAME, GNUPG_MODULE_NAME_DIRMNGR, - NULL, GPGCONF_NAME "-" DIRMNGR_NAME ".conf" }, + dirmngr_runtime_change, GPGCONF_NAME "-" DIRMNGR_NAME ".conf" }, { DIRMNGR_DISP_NAME " LDAP Server List", NULL, 0, NULL, "ldapserverlist-file", "LDAP Server" }, { "Pinentry", "pinentry", GNUPG_MODULE_NAME_PINENTRY, @@ -1064,19 +1065,20 @@ gpg_agent_runtime_change (int killflag) { gpg_error_t err; const char *pgmname; - const char *argv[2]; + const char *argv[3]; pid_t pid; pgmname = gnupg_module_name (GNUPG_MODULE_NAME_CONNECT_AGENT); - argv[0] = killflag? "KILLAGENT" : "RELOADAGENT"; - argv[1] = NULL; + argv[0] = "--no-autostart"; + argv[1] = killflag? "KILLAGENT" : "RELOADAGENT"; + argv[2] = NULL; err = gnupg_spawn_process_fd (pgmname, argv, -1, -1, -1, &pid); if (!err) err = gnupg_wait_process (pgmname, pid, 1, NULL); if (err) - gc_error (0, 0, "error running '%s%s': %s", - pgmname, " reloadagent", gpg_strerror (err)); + gc_error (0, 0, "error running '%s %s': %s", + pgmname, argv[1], gpg_strerror (err)); gnupg_release_process (pid); } @@ -1086,7 +1088,7 @@ scdaemon_runtime_change (int killflag) { gpg_error_t err; const char *pgmname; - const char *argv[6]; + const char *argv[7]; pid_t pid; (void)killflag; /* For scdaemon kill and reload are synonyms. */ @@ -1098,18 +1100,43 @@ scdaemon_runtime_change (int killflag) pgmname = gnupg_module_name (GNUPG_MODULE_NAME_CONNECT_AGENT); argv[0] = "-s"; - argv[1] = "GETINFO scd_running"; - argv[2] = "/if ${! $?}"; - argv[3] = "scd killscd"; - argv[4] = "/end"; - argv[5] = NULL; + argv[1] = "--no-autostart"; + argv[2] = "GETINFO scd_running"; + argv[3] = "/if ${! $?}"; + argv[4] = "scd killscd"; + argv[5] = "/end"; + argv[6] = NULL; + + err = gnupg_spawn_process_fd (pgmname, argv, -1, -1, -1, &pid); + if (!err) + err = gnupg_wait_process (pgmname, pid, 1, NULL); + if (err) + gc_error (0, 0, "error running '%s %s': %s", + pgmname, argv[4], gpg_strerror (err)); + gnupg_release_process (pid); +} + + +static void +dirmngr_runtime_change (int killflag) +{ + gpg_error_t err; + const char *pgmname; + const char *argv[4]; + pid_t pid; + + pgmname = gnupg_module_name (GNUPG_MODULE_NAME_CONNECT_AGENT); + argv[0] = "--no-autostart"; + argv[1] = "--dirmngr"; + argv[2] = killflag? "KILLDIRMNGR" : "RELOADDIRMNGR"; + argv[3] = NULL; err = gnupg_spawn_process_fd (pgmname, argv, -1, -1, -1, &pid); if (!err) err = gnupg_wait_process (pgmname, pid, 1, NULL); if (err) - gc_error (0, 0, "error running '%s%s': %s", - pgmname, " scd killscd", gpg_strerror (err)); + gc_error (0, 0, "error running '%s %s': %s", + pgmname, argv[2], gpg_strerror (err)); gnupg_release_process (pid); } |