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

1 files changed, 3 insertions, 1 deletions

diff --git a/server/providers/krb5/krb5_auth.c b/server/providers/krb5/krb5_auth.c
index 0321ce820..4581278de 100644
--- a/server/providers/krb5/krb5_auth.c
+++ b/server/providers/krb5/krb5_auth.c
@@ -975,10 +975,12 @@ static void krb5_child_done(struct tevent_req *req)
 
     if (*msg_status == PAM_AUTHINFO_UNAVAIL) {
         if (kr->srv != NULL) {
-            fo_set_server_status(kr->srv, SERVER_NOT_WORKING);
+            fo_set_port_status(kr->srv, PORT_NOT_WORKING);
         }
         be_mark_offline(be_req->be_ctx);
         kr->is_offline = true;
+    } else if (kr->srv != NULL) {
+        fo_set_port_status(kr->srv, PORT_WORKING);
     }
 
     struct sysdb_attrs *attrs;