sys64738
/
blackmagic
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

adiv5.c: Fix another leak.

This commit is contained in:
Uwe Bonnes 2020-04-25 14:36:06 +02:00
parent 783ec377d9
commit b0cf7d24bd
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/target/adiv5.c
View File

@ -317,9 +317,11 @@ static bool adiv5_component_probe(ADIv5_AP_t *ap, uint32_t addr, int recursion,
#endif #endif
for (int i = 0; i < 960; i++) { for (int i = 0; i < 960; i++) {
adiv5_dp_error(ap->dp);
uint32_t entry = adiv5_mem_read32(ap, addr + i*4); uint32_t entry = adiv5_mem_read32(ap, addr + i*4);
if (adiv5_dp_error(ap->dp)) { if (adiv5_dp_error(ap->dp)) {
DEBUG("%sFault reading ROM table entry\n", indent); DEBUG("%sFault reading ROM table entry %d\n", indent, i);
break;
} }
if (entry == 0) if (entry == 0)
@ -518,6 +520,7 @@ void adiv5_dp_init(ADIv5_DP_t *dp)
if (ap->base == last_base) { if (ap->base == last_base) {
DEBUG("AP %d: Duplicate base\n", i); DEBUG("AP %d: Duplicate base\n", i);
adiv5_ap_cleanup(i); adiv5_ap_cleanup(i);
free(ap);
/* FIXME: Should we expect valid APs behind duplicate ones? */ /* FIXME: Should we expect valid APs behind duplicate ones? */
return; return;
} }