summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMohammed Junaid <junaid@redhat.com>2012-01-24 17:12:59 +0530
committerVijay Bellur <vijay@gluster.com>2012-01-27 04:28:36 -0800
commite2303e774e1e37cd9becb672a67278502724b369 (patch)
tree7cade7dd58be950011b6ed8222d6f734cd4cb8b6
parente99cb60af1e153efd616014da6a54d2f95c119d1 (diff)
features/quota: Check for error in statfs fop call back.
Change-Id: Ia73d2b77dcb0507a7b8362c336c71301526aaf82 BUG: 783927 Signed-off-by: Mohammed Junaid <junaid@redhat.com> Reviewed-on: http://review.gluster.com/2685 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Jeff Darcy <jdarcy@redhat.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
-rw-r--r--xlators/features/quota/src/quota.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/xlators/features/quota/src/quota.c b/xlators/features/quota/src/quota.c
index 79172999ac7..a02ec2a79fa 100644
--- a/xlators/features/quota/src/quota.c
+++ b/xlators/features/quota/src/quota.c
@@ -2732,6 +2732,11 @@ quota_statfs_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
int64_t avail = -1;
root_inode = cookie;
+
+ /* This fop will fail mostly in case of client disconnect's,
+ * which is already logged. Hence, not logging here */
+ if (op_ret == -1)
+ goto unwind;
/*
* We should never get here unless quota_statfs (below) sent us a
* cookie, and it would only do so if the value was non-NULL. This