Commit f42e4b33 authored by Colin Ian King's avatar Colin Ian King Committed by Ben Skeggs

drm/nouveau/nouveau: fix incorrect sizeof on args.src an args.dst

The sizeof is currently on args.src and args.dst and should be on
*args.src and *args.dst. Fortunately these sizes just so happen
to be the same size so it worked, however, this should be fixed
and it also cleans up static analysis warnings

Addresses-Coverity: ("sizeof not portable")
Fixes: f268307e ("nouveau: simplify nouveau_dmem_migrate_vma")
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarBen Skeggs <bskeggs@redhat.com>
parent 48140495
...@@ -635,10 +635,10 @@ nouveau_dmem_migrate_vma(struct nouveau_drm *drm, ...@@ -635,10 +635,10 @@ nouveau_dmem_migrate_vma(struct nouveau_drm *drm,
unsigned long c, i; unsigned long c, i;
int ret = -ENOMEM; int ret = -ENOMEM;
args.src = kcalloc(max, sizeof(args.src), GFP_KERNEL); args.src = kcalloc(max, sizeof(*args.src), GFP_KERNEL);
if (!args.src) if (!args.src)
goto out; goto out;
args.dst = kcalloc(max, sizeof(args.dst), GFP_KERNEL); args.dst = kcalloc(max, sizeof(*args.dst), GFP_KERNEL);
if (!args.dst) if (!args.dst)
goto out_free_src; goto out_free_src;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment