[PATCH] mount.nfs: ensure AF priority when mounting

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx>
---
 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
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux