From 370197f6e8413c0c4c13571f1c5b613bfa1e50d5 Mon Sep 17 00:00:00 2001 From: Niels de Vos Date: Thu, 21 Jul 2016 08:12:04 +0200 Subject: md-cache: fix indention to silence Coverity Coverity complains about the incorrect indention: *** CID 1357867: Control flow issues (NESTING_INDENT_MISMATCH) ... 2566 if (conf->mdc_invalidation) 2567 ret = mdc_invalidate (this, data); >>> CID 1357867: Control flow issues (NESTING_INDENT_MISMATCH) >>> This 'if' statement is indented to column 25, as if it were nested within the preceding parent statement, but it is not. 2568 if (default_notify (this, event, data) != 0) 2569 ret = -1; 2570 break; ... Even when md-cache does not have cache-invalidation on, we need to pass the upcall to the next xlator. Change-Id: I6d2a174eb54e3df270920aae9673b5010c235f25 CID: 1357867 BUG: 1211863 Signed-off-by: Niels de Vos Reviewed-on: http://review.gluster.org/14971 Smoke: Gluster Build System Reviewed-by: Prashanth Pai Reviewed-by: Poornima G Reviewed-by: Raghavendra G CentOS-regression: Gluster Build System NetBSD-regression: NetBSD Build System Reviewed-by: Vijay Bellur --- xlators/performance/md-cache/src/md-cache.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'xlators/performance') diff --git a/xlators/performance/md-cache/src/md-cache.c b/xlators/performance/md-cache/src/md-cache.c index 57a1a0d1a4e..be42bf0a885 100644 --- a/xlators/performance/md-cache/src/md-cache.c +++ b/xlators/performance/md-cache/src/md-cache.c @@ -2565,8 +2565,8 @@ notify (xlator_t *this, int event, void *data, ...) case GF_EVENT_UPCALL: if (conf->mdc_invalidation) ret = mdc_invalidate (this, data); - if (default_notify (this, event, data) != 0) - ret = -1; + if (default_notify (this, event, data) != 0) + ret = -1; break; default: ret = default_notify (this, event, data); -- cgit