On Fri, 2013-12-13 at 15:58 -0800, Andy Grover wrote: > Allocating an array of pointers, not the objects themselves. These two > sites now match all the other sites. > > Signed-off-by: Andy Grover <agrover@xxxxxxxxxx> > --- > drivers/target/target_core_configfs.c | 2 +- > drivers/target/target_core_fabric_configfs.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > Applied to for-next. --nab > diff --git a/drivers/target/target_core_configfs.c b/drivers/target/target_core_configfs.c > index 272755d..a1c23d1 100644 > --- a/drivers/target/target_core_configfs.c > +++ b/drivers/target/target_core_configfs.c > @@ -2937,7 +2937,7 @@ static int __init target_core_init_configfs(void) > * and ALUA Logical Unit Group and Target Port Group infrastructure. > */ > target_cg = &subsys->su_group; > - target_cg->default_groups = kmalloc(sizeof(struct config_group) * 2, > + target_cg->default_groups = kmalloc(sizeof(struct config_group *) * 2, > GFP_KERNEL); > if (!target_cg->default_groups) { > pr_err("Unable to allocate target_cg->default_groups\n"); > diff --git a/drivers/target/target_core_fabric_configfs.c b/drivers/target/target_core_fabric_configfs.c > index fdadc4d..7de9f04 100644 > --- a/drivers/target/target_core_fabric_configfs.c > +++ b/drivers/target/target_core_fabric_configfs.c > @@ -906,7 +906,7 @@ static struct config_group *target_fabric_make_lun( > lun_cg->default_groups[1] = NULL; > > port_stat_grp = &lun->port_stat_grps.stat_group; > - port_stat_grp->default_groups = kzalloc(sizeof(struct config_group) * 4, > + port_stat_grp->default_groups = kzalloc(sizeof(struct config_group *) * 4, > GFP_KERNEL); > if (!port_stat_grp->default_groups) { > pr_err("Unable to allocate port_stat_grp->default_groups\n"); -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html