- Cancel the function that frees rinfo copies in case of runQueue creation error (do it inline instead)

Author: alonre24

src/dag.c

@@ -1274,7 +1274,9 @@ static bool DAG_InsertDAGToQueue(RedisAI_RunInfo *rinfo) {
             // A device run queue was not created properly, so we free everything,
             // set an error and finish.
             array_free(devices);
-            RAI_DagRunInfoFreeAllCopies(rinfo_copies);
+            for (int j = 0; j < ndevices; j++) {
+                RAI_DagRunInfoFreeShallowCopy(rinfo_copies[j]);
+            }
             array_free(rinfo_copies);
             array_free(run_queues_info);
             RAI_SetError(rinfo->err, RAI_EDAGRUN, "ERR Queue not initialized for device");

src/run_info.c

@@ -211,14 +211,6 @@ long long RAI_DagRunInfoFreeShallowCopy(RedisAI_RunInfo *rinfo) {
     return ref_count;
 }
 
-void RAI_DagRunInfoFreeAllCopies(RedisAI_RunInfo **rinfo) {
-
-    int ndevices = array_len(rinfo);
-    for (int i = 0; i < ndevices; i++) {
-        RAI_DagRunInfoFreeShallowCopy(rinfo[i]);
-    }
-}
-
 void RAI_FreeRunInfo(struct RedisAI_RunInfo *rinfo) {
     if (!rinfo) {
         return;

src/run_info.h

@@ -134,12 +134,6 @@ int RAI_ShallowCopyDagRunInfo(RedisAI_RunInfo **result, RedisAI_RunInfo *src);
  */
 long long RAI_DagRunInfoFreeShallowCopy(RedisAI_RunInfo *rinfo);
 
-/**
- * Free all the shallow copies in the RedisAI_RunInfo array.
- * @param rinfo pointers array.
- */
-void RAI_DagRunInfoFreeAllCopies(RedisAI_RunInfo **rinfo);
-
 /**
  * Frees the memory allocated on RedisAI_RunInfo
  * @param ctx Context in which Redis modules operate