diff options
| author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2012-11-28 02:49:44 +0100 |
|---|---|---|
| committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-11-28 07:49:23 +0100 |
| commit | a5675c8a8b72efdd09ccd1b0975c4670675775ef (patch) | |
| tree | ac086820e4140188a84468d8cd34133b6221c268 /drivers/target | |
| parent | target/configfs: allocate pointers instead of full struct for default_groups (diff) | |
| download | linux-a5675c8a8b72efdd09ccd1b0975c4670675775ef.tar.xz linux-a5675c8a8b72efdd09ccd1b0975c4670675775ef.zip | |
target/configfs: allocate only 6 slots for dev_cg->default_groups
Only slots 0-5 are used so 6 slots should be enough. I don't see anyone
writting anything else than NULL into ->default_groups[5] so a "late"
initialisation should not happen here.
(nab: Fix up minor apply breakage in for-next)
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target')
| -rw-r--r-- | drivers/target/target_core_configfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/target/target_core_configfs.c b/drivers/target/target_core_configfs.c index 02fd55d3a76e..f6fc258b9d40 100644 --- a/drivers/target/target_core_configfs.c +++ b/drivers/target/target_core_configfs.c @@ -2496,7 +2496,7 @@ static struct config_group *target_core_make_subdev( dev_cg = &dev->dev_group; - dev_cg->default_groups = kzalloc(sizeof(struct config_group *) * 7, + dev_cg->default_groups = kzalloc(sizeof(struct config_group *) * 6, GFP_KERNEL); if (!dev_cg->default_groups) goto out_free_device; |