On Fri 31-07-20 12:13:14, Gustavo A. R. Silva wrote: > Use the preferred form for passing the size of a structure type. The > alternative form where the structure type is spelled out hurts > readability and introduces an opportunity for a bug when the object > type is changed but the corresponding object identifier to which the > sizeof operator is applied is not. > > Signed-off-by: Gustavo A. R. Silva <gustavoars@xxxxxxxxxx> Acked-by: Michal Hocko <mhocko@xxxxxxxx> > --- > mm/memcontrol.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index bd7f972ceea4..bd0f56785841 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c > @@ -4255,7 +4255,7 @@ static int __mem_cgroup_usage_register_event(struct mem_cgroup *memcg, > new->entries[size - 1].threshold = threshold; > > /* Sort thresholds. Registering of new threshold isn't time-critical */ > - sort(new->entries, size, sizeof(struct mem_cgroup_threshold), > + sort(new->entries, size, sizeof(*new->entries), > compare_thresholds, NULL); > > /* Find current threshold */ > -- > 2.27.0 -- Michal Hocko SUSE Labs
