| commit | eb79a7bd87adc51dae77eb8d5ec37ad58ec71feb | [log] [tgz] |
|---|---|---|
| author | Mark Wielaard <mark@klomp.org> | Fri Feb 17 14:52:04 2023 +0100 |
| committer | Mark Wielaard <mark@klomp.org> | Tue Feb 21 12:57:04 2023 +0100 |
| tree | 60c0dfba1ba4edb4f6e2259e6ab7c2fa08ffafa1 | |
| parent | 77d237798c8f262d618bd3ed2db8864022bfcacb [diff] |
libasm: Fix use-after-free issue with circular single linked list cleanup
Pointed out by gcc 12 with -Wuse-after-free=3
In function ‘free_section’
asm_end.c:552:17: error: pointer ‘data’ used after ‘free’ [-Werror=use-after-free]
552 | while (oldp != scnp->content);
| ~~~~~^~~~~~~~~~~~~~~~
asm_end.c:550:9: note: call to ‘free’ here
550 | free (oldp);
| ^~~~~~~~~~~
Fix by freeing scnp->content last.
Signed-off-by: Mark Wielaard <mark@klomp.org>