"i" can be used uninitialized in one of the error branches. Fix this.

Signed-off-by: Christian König <christian.koe...@amd.com>
Reported-by: kernel test robot <l...@intel.com>
---
 drivers/dma-buf/st-dma-resv.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/dma-buf/st-dma-resv.c b/drivers/dma-buf/st-dma-resv.c
index 6f3ba756da3e..bc2265807f6c 100644
--- a/drivers/dma-buf/st-dma-resv.c
+++ b/drivers/dma-buf/st-dma-resv.c
@@ -295,7 +295,7 @@ static int test_get_fences(void *arg, bool shared)
                if (r) {
                        pr_err("Resv shared slot allocation failed\n");
                        dma_resv_unlock(&resv);
-                       goto err_free;
+                       goto err_fini;
                }
 
                dma_resv_add_shared_fence(&resv, f);
@@ -336,6 +336,7 @@ static int test_get_fences(void *arg, bool shared)
        while (i--)
                dma_fence_put(fences[i]);
        kfree(fences);
+err_fini:
        dma_resv_fini(&resv);
        dma_fence_put(f);
        return r;
-- 
2.25.1

Reply via email to