diff options
| author | Raghavendra Bhat <raghavendra@redhat.com> | 2012-10-25 17:06:31 +0530 | 
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2012-12-03 03:02:55 -0800 | 
| commit | fa1607ea596802d3d3177d775eeecb0f0e5d496f (patch) | |
| tree | a8bb59c1d459c5f6f1b90364da80b213f48d8d4f /libglusterfs | |
| parent | 3cade0a1e9dfc641e5b6db2f044af9f50ff604d6 (diff) | |
socket,rdma: before binding to any port check if it is a reserved port
* The list of ports which are reserved is obtained by reading the file
  /proc/sys/net/ipv4/ip_local_reserved_ports.
Change-Id: I3a832f53efbb79dd719b34a30cbaf28269f25816
BUG: 762989
Signed-off-by: Raghavendra Bhat <raghavendra@redhat.com>
Reviewed-on: http://review.gluster.org/4131
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'libglusterfs')
| -rw-r--r-- | libglusterfs/src/common-utils.c | 140 | ||||
| -rw-r--r-- | libglusterfs/src/common-utils.h | 4 | 
2 files changed, 144 insertions, 0 deletions
diff --git a/libglusterfs/src/common-utils.c b/libglusterfs/src/common-utils.c index 8c18b91a7b4..7a654314ee5 100644 --- a/libglusterfs/src/common-utils.c +++ b/libglusterfs/src/common-utils.c @@ -2166,3 +2166,143 @@ generate_glusterfs_ctx_id (void)          return gf_strdup (tmp_str);  } +char * +gf_get_reserved_ports () +{ +        int     proc_fd      = -1; +        char    *proc_file   = "/proc/sys/net/ipv4/ip_local_reserved_ports"; +        char    buffer[4096] = {0,}; +        char    *ports_info  = NULL; +        int32_t ret          = -1; + +        proc_fd = open (proc_file, O_RDONLY); +        if (proc_fd == -1) { +                /* What should be done in this case? error out from here +                 * and thus stop the glusterfs process from starting or +                 * continue with older method of using any of the available +                 * port? For now 2nd option is considered. +                 */ +                gf_log ("glusterfs", GF_LOG_WARNING, "could not open " +                        "the file /proc/sys/net/ipv4/ip_local_reserved_ports " +                        "for getting reserved ports info (%s)", +                        strerror (errno)); +                goto out; +        } + +        ret = read (proc_fd, buffer, sizeof (buffer)); +        if (ret < 0) { +                gf_log ("glusterfs", GF_LOG_WARNING, "could not " +                        "read the file %s for getting reserved ports " +                        "info (%s)", proc_file, strerror (errno)); +                goto out; +        } +        ports_info = gf_strdup (buffer); + +out: +        if (proc_fd != -1) +                close (proc_fd); +        return ports_info; +} + +int +gf_process_reserved_ports (gf_boolean_t *ports) +{ +        int      ret         = -1; +        char    *ports_info  = NULL; +        char    *tmp         = NULL; +        char    *blocked_port = NULL; +        gf_boolean_t  result = _gf_false; + +        ports_info = gf_get_reserved_ports (); +        if (!ports_info) { +                gf_log ("glusterfs", GF_LOG_WARNING, "Not able to get reserved " +                        "ports, hence there is a possibility that glusterfs " +                        "may consume reserved port"); +                goto out; +        } + +        blocked_port = strtok_r (ports_info, ",",&tmp); +        if (!blocked_port || !strcmp (blocked_port, ports_info)) { +                if (!blocked_port) +                        blocked_port = ports_info; +                result = gf_ports_reserved (blocked_port, ports); +                blocked_port = strtok_r (NULL, ",", &tmp); +        } + +        while (blocked_port) { +                result = gf_ports_reserved (blocked_port, ports); +                blocked_port = strtok_r (NULL, ",", &tmp); +        } + +        ret = 0; + +out: +        GF_FREE (ports_info); +        return ret; +} + +gf_boolean_t +gf_ports_reserved (char *blocked_port, gf_boolean_t *ports) +{ +        gf_boolean_t    result   = _gf_false; +        char            *range_port = NULL; +        int16_t         tmp_port1, tmp_port2 = -1; + +        if (strstr (blocked_port, "-") == NULL) { +                /* get rid of the new line character*/ +                if (blocked_port[strlen(blocked_port) -1] == '\n') +                        blocked_port[strlen(blocked_port) -1] = '\0'; +                if (gf_string2int16 (blocked_port, &tmp_port1) == 0) { +                        if (tmp_port1 > (GF_CLIENT_PORT_CEILING - 1) +                            || tmp_port1 < 0) { +                                gf_log ("glusterfs-socket", GF_LOG_WARNING, +                                        "invalid port %d", tmp_port1); +                                result = _gf_true; +                                goto out; +                        } else { +                                gf_log ("glusterfs", GF_LOG_DEBUG, +                                        "blocking port %d", tmp_port1); +                                ports[tmp_port1] = _gf_true; +                        } +                } else { +                        gf_log ("glusterfs-socket", GF_LOG_WARNING, "%s is " +                                "not a valid port identifier", blocked_port); +                        result = _gf_true; +                        goto out; +                } +        } else { +                range_port = strtok (blocked_port, "-"); +                if (!range_port){ +                        result = _gf_true; +                        goto out; +                } +                if (gf_string2int16 (range_port, &tmp_port1) == 0) { +                        if (tmp_port1 > (GF_CLIENT_PORT_CEILING - 1)) +                                tmp_port1 = GF_CLIENT_PORT_CEILING - 1; +                        if (tmp_port1 < 0) +                                tmp_port1 = 0; +                } +                range_port = strtok (NULL, "-"); +                if (!range_port) { +                        result = _gf_true; +                        goto out; +                } +                /* get rid of the new line character*/ +                if (range_port[strlen(range_port) -1] == '\n') +                        range_port[strlen(range_port) - 1] = '\0'; +                if (gf_string2int16 (range_port, &tmp_port2) == 0) { +                        if (tmp_port2 > +                            (GF_CLIENT_PORT_CEILING - 1)) +                                tmp_port2 = GF_CLIENT_PORT_CEILING - 1; +                        if (tmp_port2 < 0) +                                tmp_port2 = 0; +                } +                gf_log ("glusterfs", GF_LOG_DEBUG, "lower: %d, higher: %d", +                        tmp_port1, tmp_port2); +                for (; tmp_port1 <= tmp_port2; tmp_port1++) +                        ports[tmp_port1] = _gf_true; +        } + +out: +        return result; +} diff --git a/libglusterfs/src/common-utils.h b/libglusterfs/src/common-utils.h index 183837b9650..6d46a997105 100644 --- a/libglusterfs/src/common-utils.h +++ b/libglusterfs/src/common-utils.h @@ -76,6 +76,7 @@ void trap (void);   * nfs port in volume status.   */  #define GF_NFS3_PORT    38467 +#define GF_CLIENT_PORT_CEILING 1024  enum _gf_boolean  { @@ -553,5 +554,8 @@ uint64_t get_mem_size ();  int gf_strip_whitespace (char *str, int len);  int gf_canonicalize_path (char *path);  char *generate_glusterfs_ctx_id (void); +char *gf_get_reserved_ports(); +int gf_process_reserved_ports (gf_boolean_t ports[]); +gf_boolean_t gf_ports_reserved (char *blocked_port, gf_boolean_t *ports);  #endif /* _COMMON_UTILS_H */  | 
