* Peng Zhang <zhangpeng.00@xxxxxxxxxxxxx> [230410 05:17]: > The original code is: > memset(pivs + tmp, 0, sizeof(unsigned long *) * (max_p - tmp)); Please don't quote the code for these changes. > > The type of variable pointed to by pivs is unsigned long, but the type > used in sizeof is a pointer type. Change it to unsigned long. > > Suggested-by: David Binderman <dcb314@xxxxxxxxxxx> > Signed-off-by: Peng Zhang <zhangpeng.00@xxxxxxxxxxxxx> Cc stable, fixes line is probably needed here. Reviewed-by: Liam R. Howlett <Liam.Howlett@xxxxxxxxxx> > --- > lib/maple_tree.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/maple_tree.c b/lib/maple_tree.c > index 88c44f6d6cee..b06fc5f19b31 100644 > --- a/lib/maple_tree.c > +++ b/lib/maple_tree.c > @@ -3255,7 +3255,7 @@ static inline void mas_destroy_rebalance(struct ma_state *mas, unsigned char end > > if (tmp < max_p) > memset(pivs + tmp, 0, > - sizeof(unsigned long *) * (max_p - tmp)); > + sizeof(unsigned long) * (max_p - tmp)); > > if (tmp < mt_slots[mt]) > memset(slots + tmp, 0, sizeof(void *) * (max_s - tmp)); > -- > 2.20.1 >
