commit fc6cef1feb6aaefdf71876119cd3000af9e3bad4
author Rohit kumar <quic_rohkumar@quicinc.com> Thu Dec 02 15:47:23 2021 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Thu Dec 16 04:23:05 2021 -0800
tree 58cfe955db03de75c58f39b7d78307b483b38d68
parent 44fc01ae5c27daf6e9fab18eef47e9b44081ab9a

service: graph: Set graph handle to NULL after free

Graph_handle is not set to NULL after free at different places.
This leads to memory access to freed memory. Set graph_handle to
NULL to fix the issue.

Change-Id: I7d3fd756ab32bf1894a5466762dccecc17c51515

service/src/session_obj.c

diff --git a/service/src/session_obj.c b/service/src/session_obj.c
index 3b681f4..8052859 100644
--- a/service/src/session_obj.c
+++ b/service/src/session_obj.c
@@ -774,10 +774,12 @@
     goto done;
 
 graph_cleanup:
-    if (opened_count == 0)
+    if (opened_count == 0) {
         graph_close(sess_obj->graph);
-    else
+        sess_obj->graph = NULL;
+    } else {
         graph_remove(sess_obj->graph, merged_metadata);
+    }
 
 close_device:
     if (aif_obj->params) {
@@ -921,6 +923,7 @@
 
 graph_cleanup:
         graph_close(sess_obj->graph);
+        sess_obj->graph = NULL;
 done:
     return ret;
 }