diff options
| author | Bill Fischofer <bill.fischofer@linaro.org> | 2015-09-03 10:16:33 -0500 |
|---|---|---|
| committer | Maxim Uvarov <maxim.uvarov@linaro.org> | 2015-09-03 19:29:03 +0300 |
| commit | 57ab1d5316774c1befc201af9fae017bd2ad5965 (patch) | |
| tree | 5488320a7779cd972f19fc18a44e9b9907e70f65 /platform | |
| parent | 18d9c3c4e75bedbbca88f96723f38999005f168e (diff) | |
linux-generic: schedule: correct schedule group handling
Correct the range check on odp_schedule_group_destroy() and the
associated handling of scheduler group names.
Signed-off-by: Bill Fischofer <bill.fischofer@linaro.org>
Reviewed-and-Tested-by: Maxim Uvarov <maxim.uvarov@linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
Diffstat (limited to 'platform')
| -rw-r--r-- | platform/linux-generic/odp_schedule.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/platform/linux-generic/odp_schedule.c b/platform/linux-generic/odp_schedule.c index dc2d75f..d2543ad 100644 --- a/platform/linux-generic/odp_schedule.c +++ b/platform/linux-generic/odp_schedule.c @@ -178,7 +178,7 @@ int odp_schedule_init_global(void) odp_spinlock_init(&sched->grp_lock); for (i = 0; i < ODP_CONFIG_SCHED_GRPS; i++) { - memset(&sched->sched_grp[i].name, 0, ODP_SCHED_GROUP_NAME_LEN); + memset(sched->sched_grp[i].name, 0, ODP_SCHED_GROUP_NAME_LEN); sched->sched_grp[i].mask = thread_sched_grp_mask(i); } @@ -685,10 +685,10 @@ int odp_schedule_group_destroy(odp_schedule_group_t group) odp_spinlock_lock(&sched->grp_lock); if (group < ODP_CONFIG_SCHED_GRPS && - group > _ODP_SCHED_GROUP_NAMED && + group >= _ODP_SCHED_GROUP_NAMED && sched->sched_grp[group].name[0] != 0) { odp_thrmask_zero(sched->sched_grp[group].mask); - memset(&sched->sched_grp[group].name, 0, + memset(sched->sched_grp[group].name, 0, ODP_SCHED_GROUP_NAME_LEN); ret = 0; } else { |