Messages in this thread Patch in this message | | | From | Hamo <> | Date | Tue, 8 Nov 2011 11:14:36 +0800 | Subject | [PATCH] mount.nfs: ensure AF priority when mounting |
| |
On different architectures, getaddrinfo() may return different orders of address families. This may confuse the users. This patch fixes this problem by ensuring that we first try through IPv6(if present), then through IPv4. Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=750729
Signed-off-by: Yang Bai <hamo.by@gmail.com> --- utils/mount/stropts.c | 41 +++++++++++++++++++++++++++++++++++++++++ 1 files changed, 41 insertions(+), 0 deletions(-)
diff --git a/utils/mount/stropts.c b/utils/mount/stropts.c index 4032bf3..aa5ee86 100644 --- a/utils/mount/stropts.c +++ b/utils/mount/stropts.c @@ -654,7 +654,28 @@ static int nfs_try_mount_v3v2(struct nfsmount_info *mi) struct addrinfo *ai; int ret = 0;
+ /* first, try IPv6 */ for (ai = mi->address; ai != NULL; ai = ai->ai_next) { + if (ai->ai_family != AF_INET6) + continue; + ret = nfs_do_mount_v3v2(mi, ai->ai_addr, ai->ai_addrlen); + if (ret != 0) + return ret; + + switch (errno) { + case ECONNREFUSED: + case EOPNOTSUPP: + case EHOSTUNREACH: + continue; + default: + goto out; + } + } + + /* then, try IPv4 */ + for (ai = mi->address; ai != NULL; ai = ai->ai_next) { + if (ai->ai_family != AF_INET) + continue; ret = nfs_do_mount_v3v2(mi, ai->ai_addr, ai->ai_addrlen); if (ret != 0) return ret; @@ -742,7 +763,27 @@ static int nfs_try_mount_v4(struct nfsmount_info *mi) struct addrinfo *ai; int ret = 0;
+ /* first, try IPv6 */ for (ai = mi->address; ai != NULL; ai = ai->ai_next) { + if (ai->ai_family != AF_INET6) + continue; + ret = nfs_do_mount_v4(mi, ai->ai_addr, ai->ai_addrlen); + if (ret != 0) + return ret; + + switch (errno) { + case ECONNREFUSED: + case EHOSTUNREACH: + continue; + default: + goto out; + } + } + + /* then, try IPv4 */ + for (ai = mi->address; ai != NULL; ai = ai->ai_next) { + if (ai->ai_family != AF_INET) + continue; ret = nfs_do_mount_v4(mi, ai->ai_addr, ai->ai_addrlen); if (ret != 0) return ret; -- 1.7.1
| |