#4086 possible race condition deadlock
Closed: cloned-to-github 3 years ago by pbrezina. Opened 4 years ago by pbrezina.

To reproduce, apply the following patch and run integration tests:

diff --git a/src/providers/ldap/sdap_sudo_shared.c b/src/providers/ldap/sdap_sudo_shared.c
index 6619b0136..679ec16a6 100644
--- a/src/providers/ldap/sdap_sudo_shared.c
+++ b/src/providers/ldap/sdap_sudo_shared.c
@@ -184,6 +184,8 @@ sdap_sudo_set_usn(struct sdap_server_opts *srv_opts,
         return;
     }

+    strcmp(srv_opts->max_sudo_value, "0");
+
     errno = 0;
     usn_number = strtoul(usn, &endptr, 10);
     if (errno != 0) {

sssd_be process segfaulted in CI because srv_opts->max_sudo_value was NULL. This happened every time when the process was started. Then we hit a race condition where the new sssd_be process was trying to connect to the monitor but at the same time, the monitor was ordered to shutdown by CI test.


Metadata Update from @thalman:
- Issue tagged with: Future milestone

4 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/5050

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @pbrezina:
- Issue close_status updated to: cloned-to-github
- Issue status updated to: Closed (was: Open)

3 years ago

Login to comment on this ticket.

Metadata