summaryrefslogtreecommitdiffstats
path: root/xlators/protocol/server/src
diff options
context:
space:
mode:
authorAnkitRaj <anraj@redhat.com>2017-05-29 14:12:21 +0530
committerRaghavendra G <rgowdapp@redhat.com>2017-06-05 03:27:07 +0000
commita9d3d0438ef93beb6ec3f895923db4418c0ab3df (patch)
tree11f08de0e0e9f5cbda4aca54fedb76741423e4ae /xlators/protocol/server/src
parent9aa81c15a429dc2817f0149a33b6a9e88ead8110 (diff)
dht: for many operation directory/file path is (null) in brick log
There are few operations on mount point which results to (null) error for path on brick log. This should be corrected with correct path. Change-Id: I5acf2ead5196594c76a2e526aafbfc128ece433e BUG: 1456361 Signed-off-by: ankitraj <anraj@redhat.com> Reviewed-on: https://review.gluster.org/17407 Smoke: Gluster Build System <jenkins@build.gluster.org> Tested-by: ankitraj NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
Diffstat (limited to 'xlators/protocol/server/src')
-rw-r--r--xlators/protocol/server/src/server-rpc-fops.c12
1 files changed, 8 insertions, 4 deletions
diff --git a/xlators/protocol/server/src/server-rpc-fops.c b/xlators/protocol/server/src/server-rpc-fops.c
index 3b7f4e61f11..4e71715bf8e 100644
--- a/xlators/protocol/server/src/server-rpc-fops.c
+++ b/xlators/protocol/server/src/server-rpc-fops.c
@@ -427,7 +427,8 @@ server_access_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
op_errno, PS_MSG_ACCESS_INFO,
"%"PRId64": ACCESS %s (%s), client: %s, "
"error-xlator: %s", frame->root->unique,
- state->loc.path, uuid_utoa (state->resolve.gfid),
+ (state->loc.path) ? state->loc.path : "",
+ uuid_utoa (state->resolve.gfid),
STACK_CLIENT_NAME (frame->root),
STACK_ERR_XL_NAME (frame->root));
goto out;
@@ -1106,7 +1107,8 @@ server_symlink_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
if (op_ret < 0) {
gf_msg (this->name, GF_LOG_INFO, op_errno, PS_MSG_LINK_INFO,
"%"PRId64": SYMLINK %s (%s/%s), client: %s, "
- "error-xlator:%s", frame->root->unique, state->loc.path,
+ "error-xlator:%s", frame->root->unique,
+ (state->loc.path) ? state->loc.path : "",
uuid_utoa (state->resolve.pargfid),
state->resolve.bname,
STACK_CLIENT_NAME (frame->root),
@@ -1650,7 +1652,8 @@ server_stat_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
gf_msg (this->name, fop_log_level (GF_FOP_STAT, op_errno),
op_errno, PS_MSG_STAT_INFO,
"%"PRId64": STAT %s (%s), client: %s, error-xlator: %s",
- frame->root->unique, state->loc.path,
+ frame->root->unique,
+ (state->loc.path) ? state->loc.path : "",
uuid_utoa (state->resolve.gfid),
STACK_CLIENT_NAME (frame->root),
STACK_ERR_XL_NAME (frame->root));
@@ -1689,7 +1692,8 @@ server_setattr_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
gf_msg (this->name, GF_LOG_INFO, op_errno, PS_MSG_SETATTR_INFO,
"%"PRId64": SETATTR %s (%s), client: %s, "
"error-xlator: %s", frame->root->unique,
- state->loc.path, uuid_utoa (state->resolve.gfid),
+ (state->loc.path) ? state->loc.path : "",
+ uuid_utoa (state->resolve.gfid),
STACK_CLIENT_NAME (frame->root),
STACK_ERR_XL_NAME (frame->root));
goto out;