Bug 971276
Summary: | Client logs are filled with gf_ports_reserved warnings for port 54321 | |||
---|---|---|---|---|
Product: | [Red Hat Storage] Red Hat Gluster Storage | Reporter: | Rahul Hinduja <rhinduja> | |
Component: | glusterfs | Assignee: | Nagaprasad Sathyanarayana <nsathyan> | |
Status: | CLOSED EOL | QA Contact: | storage-qa-internal <storage-qa-internal> | |
Severity: | medium | Docs Contact: | ||
Priority: | medium | |||
Version: | 2.1 | CC: | amukherj, cbuissar, nsathyan, rhs-bugs, rwheeler, smohan, vagarwal, vbellur | |
Target Milestone: | --- | |||
Target Release: | --- | |||
Hardware: | x86_64 | |||
OS: | Linux | |||
Whiteboard: | ||||
Fixed In Version: | Doc Type: | Bug Fix | ||
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 1083089 (view as bug list) | Environment: | ||
Last Closed: | 2015-12-03 17:12:13 UTC | Type: | Bug | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | ||||
Bug Blocks: | 1050219, 1083089 |
Description
Rahul Hinduja
2013-06-06 07:39:20 UTC
Dev ack to 3.0 RHS BZs I have seen seen incident on some unrelated incident. What is the current status ? My understanding of this message is the following : Glusterfs checks for reserved ports in /proc/sys/net/ipv4/ip_local_reserved_ports. However, when it translates the string to integers, it makes use of gf_string2int16 function, which is checking for a signed 16 bits integers (between INT16_MIN and INT16_MAX [32767]). This means that inserting any port in the reserved list above INT16_MAX will lead to a warning. ==> Steps to Reproduce: 1. echo 54321 > /proc/sys/net/ipv4/ip_local_reserved_ports 2. use gluster (stopping volume was sufficient for me) 3. Look for the following warning in the logs : W [common-utils.c:2513:gf_ports_reserved] 0-glusterfs-socket: 54321 is not a valid port identifier Actual results: Port will not be taken into account Expected results: Port should not be used accidentally by gluster. Additional info: I have not tried the patch yet, but in ./libglusterfs/src/common-utils.c , gf_ports_reserved (char *blocked_port, gf_boolean_t *ports), the function used to make string to integer translation is gf_string2int16. However, using gf_string2uint16 (from 0 to UINT16_MAX/65535) is probably more suitable here. Thank you for submitting this issue for consideration in Red Hat Gluster Storage. The release for which you requested us to review, is now End of Life. Please See https://access.redhat.com/support/policy/updates/rhs/ If you can reproduce this bug against a currently maintained version of Red Hat Gluster Storage, please feel free to file a new report against the current release. |