On 04/09/2018 09:25 PM, Zhipeng Xie wrote: > From: Zhipeng Xie <xiezhipeng1@xxxxxxxxxx> > > e->percent access the mdstat_ent which was already freed in free_mdstat > > Signed-off-by: Zhipeng Xie <xiezhipeng1@xxxxxxxxxx> > --- > Detail.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Nice catch, but in the future please make sure to CC linux-raid on these postings. Applied! Thanks, Jes > diff --git a/Detail.c b/Detail.c > index 4dcf81d..860241c 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); > > -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html