Re: [PATCH 24/54] md/raid1: Improve another size determination in setup_conf()
From: Dan Carpenter
Date: Wed Oct 12 2016 - 04:29:51 EST
Compare:
foo = kmalloc(sizeof(*foo), GFP_KERNEL);
This says you are allocating enough space for foo. It can be reviewed
by looking at one line. If you change the type of foo it will still
work.
foo = kmalloc(sizeof(struct whatever), GFP_KERNEL);
There isn't enough information to say if this is correct. If you change
the type of foo then you have to update the allocation as well.
It's not a super common type of bug, but I see it occasionally.
regards,
dan carpenter