From a7525c507eb682d317dafaa7d06ba59b2c50048d Mon Sep 17 00:00:00 2001 From: Pranith Kumar K Date: Fri, 13 Apr 2018 00:16:22 +0530 Subject: cluster/afr: Make sure latency-arg is passed to afr xlator_notify doesn't pass the extra arguments that come in the input function, so XLATOR_NOTIFY macro should be used instead to pass the extra arguments to the function. BUG: 1567881 fixes bz#1567881 Change-Id: Ic15b6c446638cbacf3149693147a754219037c47 Signed-off-by: Pranith Kumar K --- libglusterfs/src/defaults-tmpl.c | 3 ++- libglusterfs/src/xlator.h | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) (limited to 'libglusterfs/src') diff --git a/libglusterfs/src/defaults-tmpl.c b/libglusterfs/src/defaults-tmpl.c index b9f6274800e..d1e919aebc8 100644 --- a/libglusterfs/src/defaults-tmpl.c +++ b/libglusterfs/src/defaults-tmpl.c @@ -122,6 +122,7 @@ default_release (xlator_t *this, fd_t *fd) int default_notify (xlator_t *this, int32_t event, void *data, ...) { + GF_UNUSED int ret = 0; switch (event) { case GF_EVENT_PARENT_UP: case GF_EVENT_PARENT_DOWN: @@ -179,7 +180,7 @@ default_notify (xlator_t *this, int32_t event, void *data, ...) while (parent) { if (parent->xlator->init_succeeded) - xlator_notify (parent->xlator, event, + XLATOR_NOTIFY (ret, parent->xlator, event, this, data); parent = parent->next; } diff --git a/libglusterfs/src/xlator.h b/libglusterfs/src/xlator.h index be6cf60e8c7..f41ebddd9a9 100644 --- a/libglusterfs/src/xlator.h +++ b/libglusterfs/src/xlator.h @@ -1135,7 +1135,7 @@ typedef struct { #define xlator_has_parent(xl) (xl->parents != NULL) -#define XLATOR_NOTIFY(_xl, params ...) \ +#define XLATOR_NOTIFY(ret, _xl, params ...) \ do { \ xlator_t *_old_THIS = NULL; \ \ -- cgit