glusterfs.git/libglusterfs/src/common-utils.c, branch v3.4.1rc1

libglusterfs: avoid the logging which says the port is invalid

2013-05-09T03:10:18+00:00

If the reserved ports file in proc contains just a newline, then
do not proceed with ports checking and reserving.

Change-Id: I776d0be1c3824dcd982f0685b171f2172b4e11e6
BUG: 762989
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/4821
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterd: harden 'volume start' staging to check for brick dirs' presence

2013-02-09T03:13:36+00:00

PROBLEM:

When the brick directory of a volume is absent on any of the servers,
AND an attempt is made to start the volume, commit fails ONLY on the
node where the brick dir is absent, leading to a split-brain like
situation.

FIX:

Harden 'volume start' to check for the presence of brick directories
at the time of staging, thereby preventing commit failure.

Change-Id: I67faeb9afbd3aa76f08645924462db126bf7a977
BUG: 889996
Signed-off-by: Krutika Dhananjay 
Reviewed-on: http://review.gluster.org/4365
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

rpc: get hostnames of client to allow FQDN based authentication

2013-02-07T00:15:54+00:00

If FQDNs are used to authenticate clients, then from
this commit forth, the client ip(v4,6) is reverse looked up
using getnameinfo to get a hostname associated with it,
if any, thereby making FQDN-based rpc authentication possible.

Change-Id: I4c5241e7079a2560de79ca15f611e65c0b858f9b
BUG: 903553
Signed-off-by: Rajesh Amaravathi 
Reviewed-on: http://review.gluster.org/4439
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

Spurious reserved port warning fix

2012-12-13T20:33:18+00:00

reserved ports is a Linux specitic concept. Make no warnings for
other systems.

BUG: 815227
Signed-off-by: Emmanuel Dreyfus 
Change-Id: I40037b9e99e7fcbdd0d153865ee7c2ef3733132e
Reviewed-on: http://review.gluster.org/4308
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

glusterd: fail vol set when value = empty string/string with all whitespaces

2012-12-10T22:18:17+00:00

PROBLEM:

'volume set' operation accepts empty strings and strings containing
all whitespaces for value. The result - subsequent attempts to start
glusterd fail.

FIX:

Added relevant checks in xlator_option_validate_* functions in options.c
and in conversion functions in common-utils.c to invalidate wrong option
values.

Change-Id: I33232c6b42ab4fcce85c2d0e0b0da145fc9232c3
BUG: 859927
Signed-off-by: Krutika Dhananjay 
Reviewed-on: http://review.gluster.org/4033
Reviewed-by: Pranith Kumar Karampuri 
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

rpc: check the ctx->listener before accessing rpcsvc object

2012-12-05T00:28:03+00:00

Change-Id: I3c913d1fab07deb41eec05c5adc29a3964315b43
BUG: 858487
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/4230
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

libglusterfs: fix unused-but-set-variable warning

2012-12-04T09:41:49+00:00

Change-Id: Ie152169a70eaec9554382b39d43ae7ffa04f041c
BUG: 762989
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/4264
Reviewed-by: Raghavendra Bhat 
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

socket,rdma: before binding to any port check if it is a reserved port

2012-12-03T11:02:55+00:00

* 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 
Reviewed-on: http://review.gluster.org/4131
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

libglusterfs: Implement float percentage

2012-11-24T07:13:39+00:00

Change-Id: Ia7ea63471f0bbd74686873f5f6f183475880f1a0
BUG: 839595
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/4162
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

In some cases AI_ADDRCONFIG confuses NetBSD 6.0

2012-10-20T07:32:58+00:00

BUG: 764655
Change-Id: I29198c94c6788f9e01f57e11798f3d3d7cd9c8de
Signed-off-by: Emmanuel Dreyfus 
Reviewed-on: http://review.gluster.org/4105
Reviewed-by: Anand Avati 
Tested-by: Anand Avati