Skip to content

Commit d4b3b4e

Browse files
committed
target: free private_config if target initialisation fails
Fixes private_config memory leak when xx_deinit_target() is not called Signed-off-by: Tomas Vanek <[email protected]> Change-Id: Ie7cce7f24af24695e7d2c1cd1882474c6863b80d Reviewed-on: https://review.openocd.org/c/openocd/+/8642 Tested-by: jenkins Reviewed-by: zapb <[email protected]> Reviewed-by: Evgeniy Naydanov <[email protected]>
1 parent cf115c1 commit d4b3b4e

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/target/target.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5848,6 +5848,7 @@ static int target_create(struct jim_getopt_info *goi)
58485848
free(target->gdb_port_override);
58495849
free(target->trace_info);
58505850
free(target->type);
5851+
free(target->private_config);
58515852
free(target);
58525853
return e;
58535854
}
@@ -5865,6 +5866,7 @@ static int target_create(struct jim_getopt_info *goi)
58655866
free(target->gdb_port_override);
58665867
free(target->trace_info);
58675868
free(target->type);
5869+
free(target->private_config);
58685870
free(target);
58695871
return JIM_ERR;
58705872
}
@@ -5878,6 +5880,7 @@ static int target_create(struct jim_getopt_info *goi)
58785880
free(target->gdb_port_override);
58795881
free(target->trace_info);
58805882
free(target->type);
5883+
free(target->private_config);
58815884
free(target);
58825885
return JIM_ERR;
58835886
}

0 commit comments

Comments
 (0)