[PATCH] dmaengine: ioatdma: Fix error handling path in 'ioat_dma_self_test()'

From: Christophe JAILLET
Date: Wed Jul 19 2017 - 18:17:01 EST


If the 'memcmp' fails, free allocated resources as done in all other
error handling paths.

Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
---
Please review carefully, this patch looks "too obvious" to me!
---
drivers/dma/ioat/init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/dma/ioat/init.c b/drivers/dma/ioat/init.c
index ed8ed1192775..948fc1f8fb5c 100644
--- a/drivers/dma/ioat/init.c
+++ b/drivers/dma/ioat/init.c
@@ -390,7 +390,7 @@ static int ioat_dma_self_test(struct ioatdma_device *ioat_dma)
if (memcmp(src, dest, IOAT_TEST_SIZE)) {
dev_err(dev, "Self-test copy failed compare, disabling\n");
err = -ENODEV;
- goto free_resources;
+ goto unmap_dma;
}

unmap_dma:
--
2.11.0