Markus Elfring
2020-Jul-15 18:12 UTC
[Nouveau] [PATCH] drm/nouveau: add the missed kfree() for nouveau_bo_alloc()
> ? to fix it.I suggest to replace this wording by the tag ?Fixes?. ?> +++ b/drivers/gpu/drm/nouveau/nouveau_bo.c > @@ -276,8 +276,10 @@ nouveau_bo_alloc(struct nouveau_cli *cli, u64 *size, int *align, u32 flags, > break; > } > > - if (WARN_ON(pi < 0)) > + if (WARN_ON(pi < 0)) { > + kfree(nvbo); > return ERR_PTR(-EINVAL); > + }I propose to move such common exception handling to the end of this function implementation so that a bit of duplicate code will be avoided. https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/coding-style.rst?id=e9919e11e219eaa5e8041b7b1a196839143e9125#n481 Regards, Markus
Possibly Parallel Threads
- [PATCH] drm/nouveau: add the missed kfree() for nouveau_bo_alloc()
- [PATCH] drm/nouveau: Fix memory leak in nouveau_bo_alloc
- [PATCH] drm/nouveau: Fix memory leak in nouveau_bo_alloc
- [PATCH] drm/nouveau: Fix memory leak in nouveau_bo_alloc
- [PATCH] drm/nouveau: Fix memory leak in nouveau_bo_alloc