diff options
| author | Werner Koch <[email protected]> | 2015-06-18 11:34:48 +0000 |
|---|---|---|
| committer | Werner Koch <[email protected]> | 2015-06-18 11:34:48 +0000 |
| commit | c97198371b7307e64afdd323231977b2247f64ec (patch) | |
| tree | 87f9336039fc61fee97c5c7df8cef6f19ba74ac8 /dirmngr/ldap-wrapper.c | |
| parent | agent: Print a warning for obsolete options. (diff) | |
| download | gnupg-c97198371b7307e64afdd323231977b2247f64ec.tar.gz gnupg-c97198371b7307e64afdd323231977b2247f64ec.zip | |
dirmngr: Fix the cleanup zombies fix (685b782).
* dirmngr/ldap-wrapper.c (ldap_wrapper_thread): Do not close the
stdout reader after EOF from read_log_data.
* dirmngr/crlcache.c (crl_cache_reload_crl): Close the reader before
the next iteration.
--
I assumed that the log_fd also has a reader object but that reader
object is used for stdout and needs to be closed by the consumer.
The real bug with the non-released ldap_wrapper control objects was
that when looping over distribution points we did not closed the used
reader object before the next iteration. Now, the test case had more
than one DP and thus we lost one reader object.
Signed-off-by: Werner Koch <[email protected]>
Diffstat (limited to 'dirmngr/ldap-wrapper.c')
| -rw-r--r-- | dirmngr/ldap-wrapper.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/dirmngr/ldap-wrapper.c b/dirmngr/ldap-wrapper.c index 0dcc7baf5..a54e40523 100644 --- a/dirmngr/ldap-wrapper.c +++ b/dirmngr/ldap-wrapper.c @@ -325,8 +325,6 @@ ldap_wrapper_thread (void *dummy) { if (read_log_data (ctx)) { - ksba_reader_release (ctx->reader); - ctx->reader = NULL; SAFE_CLOSE (ctx->log_fd); any_action = 1; } |