aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/md/raid5.c
diff options
context:
space:
mode:
authorNeilBrown <neilb@suse.de>2013-11-24 19:12:43 -0500
committerNeilBrown <neilb@suse.de>2013-11-27 19:00:04 -0500
commit0c775d5208284700de423e6746259da54a42e1f5 (patch)
treef32462cf475fb8a9a4fbe395c7016086159e1a43 /drivers/md/raid5.c
parent6ce4eac1f600b34f2f7f58f9cd8f0503d79e42ae (diff)
md/raid5: fix new memory-reference bug in alloc_thread_groups.
In alloc_thread_groups, worker_groups is a pointer to an array, not an array of pointers. So worker_groups[i] is wrong. It should be &(*worker_groups)[i] Found-by: coverity Fixes: 60aaf9338545 Reported-by: Ben Hutchings <bhutchings@solarflare.com> Cc: majianpeng <majianpeng@gmail.com> Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'drivers/md/raid5.c')
-rw-r--r--drivers/md/raid5.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c
index 47da0af6322b..676d8b7c5117 100644
--- a/drivers/md/raid5.c
+++ b/drivers/md/raid5.c
@@ -5471,7 +5471,7 @@ static int alloc_thread_groups(struct r5conf *conf, int cnt,
5471 for (i = 0; i < *group_cnt; i++) { 5471 for (i = 0; i < *group_cnt; i++) {
5472 struct r5worker_group *group; 5472 struct r5worker_group *group;
5473 5473
5474 group = worker_groups[i]; 5474 group = &(*worker_groups)[i];
5475 INIT_LIST_HEAD(&group->handle_list); 5475 INIT_LIST_HEAD(&group->handle_list);
5476 group->conf = conf; 5476 group->conf = conf;
5477 group->workers = workers + i * cnt; 5477 group->workers = workers + i * cnt;