summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZhipeng Xie <xiezhipeng1@huawei.com>2018-04-10 09:25:39 +0800
committerDimitri John Ledkov <xnox@ubuntu.com>2018-05-15 12:10:33 +0100
commit9007bb8765da1df600944c4bcd43a404334ddc82 (patch)
tree157d17fed7902a18dc5a0cfbfb359c3dc30bea44
parent7a53226e67eee2357abb918766f38b3cba6f625f (diff)
[PATCH 1/9] mdadm: fix use-after-free after free_mdstat
e->percent access the mdstat_ent which was already freed in free_mdstat Signed-off-by: Zhipeng Xie <xiezhipeng1@huawei.com> Signed-off-by: Jes Sorensen <jsorensen@fb.com> Gbp-Pq: Name 0001-mdadm-fix-use-after-free-after-free_mdstat.patch
-rw-r--r--Detail.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/Detail.c b/Detail.c
index 4dcf81dd..860241ce 100644
--- a/Detail.c
+++ b/Detail.c
@@ -561,7 +561,6 @@ int Detail(char *dev, struct context *c)
printf(" %7s Status : %d%% complete\n",
sync_action[e->resync], e->percent);
}
- free_mdstat(ms);
if ((st && st->sb) && (info && info->reshape_active)) {
#if 0
@@ -609,6 +608,8 @@ This is pretty boring
printf("\n");
} else if (e && e->percent >= 0)
printf("\n");
+ free_mdstat(ms);
+
if (st && st->sb)
st->ss->detail_super(st, c->homehost);