74b1de
From 769263ad422e3c1069de0994ff2274044982b242 Mon Sep 17 00:00:00 2001
74b1de
From: Mohit Agrawal <moagrawal@redhat.com>
74b1de
Date: Sun, 1 Sep 2019 12:01:09 +0530
74b1de
Subject: [PATCH 292/297] rpc: Update address family if it is not provide in
74b1de
 cmd-line arguments
74b1de
74b1de
Problem: After enabling transport-type to inet6 and passed ipv6
74b1de
         transport.socket.bind-address in glusterd.vol clients are
74b1de
         not started.
74b1de
74b1de
Solution: Need to update address-family based on remote-address for
74b1de
          all gluster client process
74b1de
74b1de
> Change-Id: Iaa3588cd87cebc45231bfd675745c1a457dc9b31
74b1de
> Fixes: bz#1747746
74b1de
> Credits: Amgad Saleh <amgad.saleh@nokia.com>
74b1de
> Signed-off-by: Mohit Agrawal <moagrawal@redhat.com>
74b1de
> (Cherry picked from commit 80b8cfe3f1386606bada97a76a0cad7acdf6b877)
74b1de
> (Reviewed on upstream link https://review.gluster.org/#/c/glusterfs/+/23340/)
74b1de
74b1de
Change-Id: Iaa3588cd87cebc45231bfd675745c1a457dc9b31
74b1de
BUG: 1750241
74b1de
Signed-off-by: Mohit Agrawal <moagrawal@redhat.com>
74b1de
Reviewed-on: https://code.engineering.redhat.com/gerrit/181184
74b1de
Tested-by: Mohit Agrawal <moagrawa@redhat.com>
74b1de
Tested-by: RHGS Build Bot <nigelb@redhat.com>
74b1de
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
74b1de
---
74b1de
 rpc/rpc-transport/socket/src/name.c | 13 ++++++++++++-
74b1de
 1 file changed, 12 insertions(+), 1 deletion(-)
74b1de
74b1de
diff --git a/rpc/rpc-transport/socket/src/name.c b/rpc/rpc-transport/socket/src/name.c
74b1de
index 7f18cc4..b473f3b 100644
74b1de
--- a/rpc/rpc-transport/socket/src/name.c
74b1de
+++ b/rpc/rpc-transport/socket/src/name.c
74b1de
@@ -214,6 +214,7 @@ af_inet_client_get_remote_sockaddr(rpc_transport_t *this,
74b1de
     uint16_t remote_port = 0;
74b1de
     struct addrinfo *addr_info = NULL;
74b1de
     int32_t ret = 0;
74b1de
+    struct in6_addr serveraddr;
74b1de
 
74b1de
     remote_host_data = dict_get(options, "remote-host");
74b1de
     if (remote_host_data == NULL) {
74b1de
@@ -249,6 +250,13 @@ af_inet_client_get_remote_sockaddr(rpc_transport_t *this,
74b1de
         goto err;
74b1de
     }
74b1de
 
74b1de
+    /* Need to update transport-address family if address-family is not provide
74b1de
+       to command-line arguments
74b1de
+    */
74b1de
+    if (inet_pton(AF_INET6, remote_host, &serveraddr)) {
74b1de
+        sockaddr->sa_family = AF_INET6;
74b1de
+    }
74b1de
+
74b1de
     /* TODO: gf_resolve is a blocking call. kick in some
74b1de
        non blocking dns techniques */
74b1de
     ret = gf_resolve_ip6(remote_host, remote_port, sockaddr->sa_family,
74b1de
@@ -522,7 +530,10 @@ socket_client_get_remote_sockaddr(rpc_transport_t *this,
74b1de
             ret = -1;
74b1de
     }
74b1de
 
74b1de
-    if (*sa_family == AF_UNSPEC) {
74b1de
+    /* Address-family is updated based on remote_host in
74b1de
+       af_inet_client_get_remote_sockaddr
74b1de
+    */
74b1de
+    if (*sa_family != sockaddr->sa_family) {
74b1de
         *sa_family = sockaddr->sa_family;
74b1de
     }
74b1de
 
74b1de
-- 
74b1de
1.8.3.1
74b1de