diff options
| author | Mohammed Junaid <junaid@redhat.com> | 2012-01-24 17:12:59 +0530 |
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2012-01-27 04:28:36 -0800 |
| commit | e2303e774e1e37cd9becb672a67278502724b369 (patch) | |
| tree | 7cade7dd58be950011b6ed8222d6f734cd4cb8b6 | |
| parent | e99cb60af1e153efd616014da6a54d2f95c119d1 (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.c | 5 |
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 |