diff options
author | Poornima G <pgurusid@redhat.com> | 2014-04-07 17:38:32 +0530 |
---|---|---|
committer | Niels de Vos <ndevos@redhat.com> | 2014-07-31 00:50:47 -0700 |
commit | a0cb3b0665bfbe321b8a21d609cfc2defb572084 (patch) | |
tree | bedd27bbec184d37252f9eaf9ceb7f2525123730 | |
parent | d5f72dc49604aec2643d92a1b4e321c532ef8d05 (diff) |
gfapi: In glfs_set_volfile_server() remove the port number check.
The documentation for glfs_set_volfile_server() api says that the
port zero is a valid value. Specifying 0 uses the default port
number GF_DEFAULT_BASE_PORT. Hence removing the check for port being
zero.
Master branch commit 6567d141c1b0112acb35e711371ae5a879fc645a:
> Change-Id: I942a7fe54a418231a438ab67756537df8f28d2de
> BUG: 1084964
> Signed-off-by: Poornima G <pgurusid@redhat.com>
> Reviewed-on: http://review.gluster.org/7409
> Reviewed-by: Raghavendra Talur <rtalur@redhat.com>
> Reviewed-by: Krishnan Parthasarathi <kparthas@redhat.com>
> Tested-by: Gluster Build System <jenkins@build.gluster.com>
> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Change-Id: I942a7fe54a418231a438ab67756537df8f28d2de
BUG: 1124728
Signed-off-by: Poornima G <pgurusid@redhat.com>
Reviewed-on: http://review.gluster.org/8374
Reviewed-by: Niels de Vos <ndevos@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
-rw-r--r-- | api/src/glfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/api/src/glfs.c b/api/src/glfs.c index 069b7aa23fa..4329cc6109a 100644 --- a/api/src/glfs.c +++ b/api/src/glfs.c @@ -356,7 +356,7 @@ glfs_set_volfile_server (struct glfs *fs, const char *transport, server_cmdline_t *tmp = NULL; int ret = -1; - if (!transport || !host || !port) { + if (!transport || !host) { errno = EINVAL; return ret; } |