diff options
| author | jiffin tony thottan <jthottan@redhat.com> | 2015-11-16 15:51:01 +0530 |
|---|---|---|
| committer | Raghavendra G <rgowdapp@redhat.com> | 2015-11-17 02:08:00 -0800 |
| commit | eada31c88f929768df5fa337137b2ffcd57f2e73 (patch) | |
| tree | 89b39f419390c301acd98c27b8590474d5a0f61e | |
| parent | b6323c1995223d3566ffca12145c943df3fdc2ed (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.c | 9 |
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); } |