Fixed memory leak issue with script_str variable

asanchez1987 · Mar 16, 2015 · 80ab6ff · 80ab6ff
1 parent dc6c5f5
commit 80ab6ff
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/plugins/jobcomp/elasticsearch/jobcomp_elasticsearch.c b/src/plugins/jobcomp/elasticsearch/jobcomp_elasticsearch.c
@@ -879,6 +879,7 @@ extern int slurm_jobcomp_log_record(struct job_record *job_ptr)
 		xrealloc(tmp, sizeof(char) * strlen(script_str) + 1 + 12);
 		sprintf(tmp, ",\"script\":\"%s\"", script_str);
 		xstrcat(buffer, tmp);
+		xfree(script_str);
 	}
 
 	if (job_ptr->assoc_ptr) {
@@ -936,7 +937,6 @@ extern int slurm_jobcomp_log_record(struct job_record *job_ptr)
 		}
 	}
 
-	xfree(script_str);
 	xfree(parent_accounts);
 	xfree(buffer);
 	xfree(tmp);