tegra: dma: fix coverity issue

In case of NULL returned from alloc, it should not passed down to
put function, where it can be dereferenced

Bug 200222365

Change-Id: I546cdedbbe1c297bb81853b955e7845d2d2085f6
(cherry picked from commit e396da726c8540d0f408c5cb31c3235ba8d5bbaa)
Signed-off-by: dmitry pervushin <dpervushin@nvidia.com>
Reviewed-on: http://git-master/r/1227729
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Sumeet Gupta <sumeetg@nvidia.com>
Tested-by: Sumeet Gupta <sumeetg@nvidia.com>

1 files changed, 3 insertions, 1 deletions

diff --git a/drivers/dma/tegra186-gpc-dma.c b/drivers/dma/tegra186-gpc-dma.c
index 4327d2305..81dc8b32b 100644
--- a/drivers/dma/tegra186-gpc-dma.c
+++ b/drivers/dma/tegra186-gpc-dma.c
@@ -401,8 +401,10 @@ static struct tegra_dma_sg_req *tegra_dma_sg_req_alloc(
 {
         struct tegra_dma_sg_req *sg_req = NULL;
         sg_req = devm_kzalloc(tdc2dev(tdc), sizeof(struct tegra_dma_sg_req), GFP_ATOMIC);
-        if (!sg_req)
+        if (!sg_req) {
                 dev_err(tdc2dev(tdc), "sg_req alloc failed\n");
+                return NULL;
+        }
         if (prealloc)
                 tegra_dma_sg_req_put(tdc, sg_req, true);
         return sg_req;