diff options
author | Martin Nagy <mnagy@redhat.com> | 2009-12-10 22:23:04 +0100 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-12-15 08:33:55 -0500 |
commit | 796a6c947951a5a4459545b99bf97e0523f71402 (patch) | |
tree | 35725ab411fd405a0f4b781e183bd7ac6d35ba73 /server/providers/fail_over.h | |
parent | c48d88aa2fc016670c30ad0a426e7e86ff9090d7 (diff) | |
download | sssd-796a6c947951a5a4459545b99bf97e0523f71402.tar.gz sssd-796a6c947951a5a4459545b99bf97e0523f71402.tar.bz2 sssd-796a6c947951a5a4459545b99bf97e0523f71402.zip |
Don't consider one address with different port numbers as the same
There were two problems with the code. We were using
fo_set_server_status() instead of fo_set_port_status() when we failed to
connect to a service. This is a problem because if two services use the
same server, or we want to use one server with two different ports,
marking the whole server as bad is incorrect. The other problem was that
be_resolve_server_done() was comparing the hostent structures -- these
are, however, equal across multiple server:port pairs with the same
server addresses.
Fixes: #321
Diffstat (limited to 'server/providers/fail_over.h')
-rw-r--r-- | server/providers/fail_over.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/server/providers/fail_over.h b/server/providers/fail_over.h index 5fa9ff0c..e581fbaf 100644 --- a/server/providers/fail_over.h +++ b/server/providers/fail_over.h @@ -88,7 +88,7 @@ void fo_set_server_status(struct fo_server *server, /* * Set feedback about the port status. This function should be used when * the server itself is working but the service is not. When status is set - * to PORT_WORKING, 'server' is also marked as an "active server" for it's + * to PORT_WORKING, 'server' is also marked as an "active server" for its * service. When the next fo_resolve_service_send() function is called, this * server will be preferred. This will hold as long as it is not marked as * not-working. |