Browse Source

media: vimc: Remove redundant free

Commit 4a29b7090749 ("[media] vimc: Subdevices as modules") removes
vimc allocation from vimc_probe(), so corresponding deallocation
on the error path tries to free static memory.

Found by Linux Driver Verification project (linuxtesting.org).

Signed-off-by: Anton Vasilyev <vasilyev@ispras.ru>
Acked-by: Helen Koike <helen.koike@collabora.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Anton Vasilyev 7 years ago
parent
commit
b9f0f653ea
1 changed files with 0 additions and 1 deletions
  1. 0 1
      drivers/media/platform/vimc/vimc-core.c

+ 0 - 1
drivers/media/platform/vimc/vimc-core.c

@@ -328,7 +328,6 @@ static int vimc_probe(struct platform_device *pdev)
 	if (ret) {
 		media_device_cleanup(&vimc->mdev);
 		vimc_rm_subdevs(vimc);
-		kfree(vimc);
 		return ret;
 	}