diff options
| author | Arto Merilainen <amerilainen@nvidia.com> | 2013-05-29 13:26:06 +0300 |
|---|---|---|
| committer | Thierry Reding <thierry.reding@gmail.com> | 2013-06-22 12:43:54 +0200 |
| commit | edeabfcbc150a48e56dd411195ef812134983d6f (patch) | |
| tree | 0c0d5c24edb5374eb483a6863213e4e59272604a /drivers/gpu/host1x/syncpt.c | |
| parent | 3364cd28906d87f0c77754998679bb66639d4112 (diff) | |
| download | op-kernel-dev-edeabfcbc150a48e56dd411195ef812134983d6f.zip op-kernel-dev-edeabfcbc150a48e56dd411195ef812134983d6f.tar.gz | |
gpu: host1x: Fix memory access in syncpt request
This patch fixes a bad memory access in syncpoint request code. If
no syncpoints were available, the code accessed unreserved memory
area causing unexpected behaviour.
Signed-off-by: Arto Merilainen <amerilainen@nvidia.com>
Acked-By: Terje Bergstrom <tbergstrom@nvidia.com>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
Diffstat (limited to 'drivers/gpu/host1x/syncpt.c')
| -rw-r--r-- | drivers/gpu/host1x/syncpt.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/gpu/host1x/syncpt.c b/drivers/gpu/host1x/syncpt.c index 4b49345..2b03f1b 100644 --- a/drivers/gpu/host1x/syncpt.c +++ b/drivers/gpu/host1x/syncpt.c @@ -40,7 +40,8 @@ static struct host1x_syncpt *_host1x_syncpt_alloc(struct host1x *host, for (i = 0; i < host->info->nb_pts && sp->name; i++, sp++) ; - if (sp->dev) + + if (i >= host->info->nb_pts) return NULL; name = kasprintf(GFP_KERNEL, "%02d-%s", sp->id, |
