soc: qcom: Clear the memory before freeing it up

It is a case of write after free, this is causing page allocation
failure due to corruption. This is due to freeing up of segments
allocated for venus subsystem, when venus fw loading fail midway.

Change-Id: I0019a05b1d1336dcf361264607597430e5f1625a
Signed-off-by: Avaneesh Kumar Dwivedi <akdwived@codeaurora.org>

diff --git a/drivers/soc/qcom/peripheral-loader.c b/drivers/soc/qcom/peripheral-loader.c
index 6e5ddc4..3415338 100644 (file)
--- a/drivers/soc/qcom/peripheral-loader.c
+++ b/drivers/soc/qcom/peripheral-loader.c
@@ -917,13 +917,13 @@ out:
                                                priv->region_start),
                                        VMID_HLOS);
                        }
+                       if (desc->clear_fw_region && priv->region_start)
+                               pil_clear_segment(desc);
                        dma_free_attrs(desc->dev, priv->region_size,
                                        priv->region, priv->region_start,
                                        &desc->attrs);
                        priv->region = NULL;
                }
-               if (desc->clear_fw_region && priv->region_start)
-                       pil_clear_segment(desc);
                pil_release_mmap(desc);
        }
        return ret;