fixed memory leak in japi

// BelongsTo: CS-582
hpc-gridware · Sep 17, 2024 · d3d48e7 · d3d48e7
1 parent 51eaea4
commit d3d48e7
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/source/libs/gdi/sge_gdi_data.cc b/source/libs/gdi/sge_gdi_data.cc
@@ -124,6 +124,14 @@ static void
 gdi_data_ts_destroy() {
    DENTER(TOP_LAYER);
    pthread_mutex_destroy(&ts.mutex);
+
+   /* @todo CS-591 don't we have to free all the other attributes as well?
+    * char *master_host;
+    * u_long64 timestamp_qmaster_file;
+    * char *ssl_private_key;
+    * char *ssl_certificate;
+    * sge_csp_path_class_t *csp_path_obj;
+    */
    DRETURN_VOID;
 }
 

diff --git a/source/libs/japi/japi.cc b/source/libs/japi/japi.cc
@@ -405,6 +405,7 @@ int japi_init_mt(dstring *diag)
       lFreeList(&alp);
       DRETURN(DRMAA_ERRNO_INTERNAL_ERROR);
    }
+   lFreeList(&alp);
 
    DRETURN(DRMAA_ERRNO_SUCCESS);
 }

diff --git a/source/libs/uti/sge_csp_path.h b/source/libs/uti/sge_csp_path.h
@@ -92,5 +92,3 @@ sge_csp_path_class_t *
 sge_csp_path_class_create(sge_error_class_t *eh);
 
 void sge_csp_path_class_destroy(sge_csp_path_class_t **pst);
-
-void sge_csp_path_class_destroy(sge_csp_path_class_t **pst);