diff options
author | Ravishankar N <ravishankar@redhat.com> | 2017-08-18 18:05:54 +0530 |
---|---|---|
committer | jiffin tony Thottan <jthottan@redhat.com> | 2017-09-07 06:46:19 +0000 |
commit | e6aaeda4c3a44f1c459adbd619ee7c6d0151b254 (patch) | |
tree | 4df780f1b30547b82c0ae25bb7184e4b908f2327 /xlators/cluster/afr/src/afr.h | |
parent | a311911dcc1e80e9c70e8a9b356b8dfc9eda1193 (diff) |
afr: check validity of afr_reply
...in various self-heal code paths.
Originally found by Pranith in __afr_selfheal_name_impunge ()
Also change __afr_selfheal_assign_gfid() to send lookup only on those
bricks that don't have a gfid matching that of the source.
> Reviewed-on: https://review.gluster.org/18065
> Smoke: Gluster Build System <jenkins@build.gluster.org>
> CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
(cherry picked from commit d594900dbca92c356152be65fce16f77c402117c)
Change-Id: I70a2ccd750a2af92c5fc36e0eefb2b6125404b4a
BUG: 1487319
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Signed-off-by: Ravishankar N <ravishankar@redhat.com>
Reviewed-on: https://review.gluster.org/18173
Smoke: Gluster Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'xlators/cluster/afr/src/afr.h')
-rw-r--r-- | xlators/cluster/afr/src/afr.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/xlators/cluster/afr/src/afr.h b/xlators/cluster/afr/src/afr.h index cf736ed3d2e..c4ceb66914f 100644 --- a/xlators/cluster/afr/src/afr.h +++ b/xlators/cluster/afr/src/afr.h @@ -1159,6 +1159,9 @@ void afr_remove_eager_lock_stub (afr_local_t *local); void +afr_reply_wipe (struct afr_reply *reply); + +void afr_replies_wipe (struct afr_reply *replies, int count); gf_boolean_t |