diff options
| author | NIIBE Yutaka <[email protected]> | 2018-09-10 00:16:50 +0000 |
|---|---|---|
| committer | NIIBE Yutaka <[email protected]> | 2018-09-10 00:22:38 +0000 |
| commit | 213379debe5591dad6339aa95aa7282e0de620f9 (patch) | |
| tree | a4cf91e4dac2e8de111460ff6fee2c4eff261981 | |
| parent | g10: Fix memory leak. (diff) | |
| download | gnupg-213379debe5591dad6339aa95aa7282e0de620f9.tar.gz gnupg-213379debe5591dad6339aa95aa7282e0de620f9.zip | |
agent: Fix error code check from npth_mutex_init.
* agent/call-pinentry.c (initialize_module_call_pinentry): It's an
error when npth_mutex_init returns non-zero.
--
Cherry-pick from master commit of:
adce73b86fd49d5bbb8884231a26cc7533d400e2
Actually, initialize_module_call_pinentry is only called once from
main. So, this bug had no harm and having the static variable
INITIALIZED is not needed.
Signed-off-by: NIIBE Yutaka <[email protected]>
| -rw-r--r-- | agent/call-pinentry.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/agent/call-pinentry.c b/agent/call-pinentry.c index a0886814f..b68d0a8c9 100644 --- a/agent/call-pinentry.c +++ b/agent/call-pinentry.c @@ -98,11 +98,15 @@ void initialize_module_call_pinentry (void) { static int initialized; + int err; if (!initialized) { - if (npth_mutex_init (&entry_lock, NULL)) - initialized = 1; + err = npth_mutex_init (&entry_lock, NULL); + if (err) + log_fatal ("error initializing mutex: %s\n", strerror (err)); + + initialized = 1; } } |