summaryrefslogtreecommitdiffstats
path: root/xlators/features/trash/src/trash.c
diff options
context:
space:
mode:
authorMohammed Rafi KC <rkavunga@redhat.com>2019-06-27 19:17:29 +0530
committerMohammed Rafi KC <rkavunga@redhat.com>2019-06-27 19:59:54 +0530
commitefa8477013d98a26283302ecb42d4ba606ce405e (patch)
treeacc8fe6f69dac4ab3de25ee5d177a8e5425a9aff /xlators/features/trash/src/trash.c
parent5d66eafec581fb3209af74595784be8854ca40a4 (diff)
downloadglusterfs-efa8477013d98a26283302ecb42d4ba606ce405e.tar.gz
glusterfs-efa8477013d98a26283302ecb42d4ba606ce405e.tar.xz
glusterfs-efa8477013d98a26283302ecb42d4ba606ce405e.zip
graph/shd: Use top down approach while cleaning xlator
We were cleaning xlator from botton to top, which might lead to problems when upper xlators trying to access the xlator object loaded below. One such scenario is when fd_unref happens as part of the fini call which might lead to calling the releasedir to lower xlator. This will lead to invalid mem access Change-Id: I8a6cb619256fab0b0c01a2d564fc88287c4415a0 Updates: bz#1716695 Signed-off-by: Mohammed Rafi KC <rkavunga@redhat.com>
Diffstat (limited to 'xlators/features/trash/src/trash.c')
-rw-r--r--xlators/features/trash/src/trash.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/xlators/features/trash/src/trash.c b/xlators/features/trash/src/trash.c
index f96ed73c10..f44b11c687 100644
--- a/xlators/features/trash/src/trash.c
+++ b/xlators/features/trash/src/trash.c
@@ -2523,6 +2523,7 @@ out:
GF_FREE(priv);
}
mem_pool_destroy(this->local_pool);
+ this->local_pool = NULL;
}
return ret;
}