diff options
author | J. Bruce Fields <bfields@redhat.com> | 2012-11-28 09:11:29 -0500 |
---|---|---|
committer | Steve Dickson <steved@redhat.com> | 2012-11-28 09:33:08 -0500 |
commit | cb3b81d6c4167ede8886115e271584a96ea55565 (patch) | |
tree | de0ff1ecec9e301705d20fba9a89f86795a2be59 | |
parent | e07913c2c5e4dc5e406c2cb12cf78150cab3d95a (diff) | |
download | nfs-utils-cb3b81d6c4167ede8886115e271584a96ea55565.tar.gz nfs-utils-cb3b81d6c4167ede8886115e271584a96ea55565.tar.xz nfs-utils-cb3b81d6c4167ede8886115e271584a96ea55565.zip |
mountd: auth_unix_ip should downcall on error to prevent hangs
Since bf6a4febaa78bf188896b7b5b02c46562dd08b70 "mountd: handle
allocation failures in auth_unix_ip upcall", a failure to map the
address of an incoming client to a name could result in a hang.
We should be responding with an error in the case, not just skipping the
downcall and leaving everybody hanging.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
-rw-r--r-- | utils/mountd/cache.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/utils/mountd/cache.c b/utils/mountd/cache.c index e950ec6..c13f305 100644 --- a/utils/mountd/cache.c +++ b/utils/mountd/cache.c @@ -109,12 +109,10 @@ static void auth_unix_ip(FILE *f) struct addrinfo *ai = NULL; ai = client_resolve(tmp->ai_addr); - if (ai == NULL) - goto out; - client = client_compose(ai); - freeaddrinfo(ai); - if (!client) - goto out; + if (ai) { + client = client_compose(ai); + freeaddrinfo(ai); + } } qword_print(f, "nfsd"); qword_print(f, ipaddr); @@ -127,7 +125,6 @@ static void auth_unix_ip(FILE *f) xlog(D_CALL, "auth_unix_ip: client %p '%s'", client, client?client: "DEFAULT"); free(client); -out: freeaddrinfo(tmp); } |