summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjiffin tony thottan <jthottan@redhat.com>2015-11-16 15:51:01 +0530
committerRaghavendra G <rgowdapp@redhat.com>2015-11-17 02:08:00 -0800
commiteada31c88f929768df5fa337137b2ffcd57f2e73 (patch)
tree89b39f419390c301acd98c27b8590474d5a0f61e
parentb6323c1995223d3566ffca12145c943df3fdc2ed (diff)
marker : handle variable `ret` properly in marker_unlink_cbk
While compiling new source, the following issue got hit : marker.c: In function 'marker_unlink_cbk': marker.c:994:29: warning: variable 'ret' set but not used int32_t ret = 0; This patch will fix the same. Change-Id: I0de60bed3351b3aa1cc80f52d178e447826210e7 BUG: 1257694 Signed-off-by: Jiffin Tony Thottan <jthottan@redhat.com> Reviewed-on: http://review.gluster.org/12584 Reviewed-by: Vijaikumar Mallikarjuna <vmallika@redhat.com> Tested-by: NetBSD Build System <jenkins@build.gluster.org> Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
-rw-r--r--xlators/features/marker/src/marker.c9
1 files changed, 7 insertions, 2 deletions
diff --git a/xlators/features/marker/src/marker.c b/xlators/features/marker/src/marker.c
index 8007933101d..77d8a89abd0 100644
--- a/xlators/features/marker/src/marker.c
+++ b/xlators/features/marker/src/marker.c
@@ -1012,10 +1012,15 @@ marker_unlink_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
if (priv->feature_enabled & GF_QUOTA) {
if (!local->skip_txn) {
- if (xdata)
+ if (xdata) {
ret = dict_get_uint32 (xdata,
GF_RESPONSE_LINK_COUNT_XDATA, &nlink);
-
+ if (ret) {
+ gf_log (this->name, GF_LOG_TRACE,
+ "dict get failed %s ",
+ strerror (-ret));
+ }
+ }
mq_reduce_parent_size_txn (this, &local->loc, NULL,
nlink);
}