diff options
author | Robin Haberkorn <robin.haberkorn@googlemail.com> | 2025-03-19 12:32:10 +0300 |
---|---|---|
committer | Robin Haberkorn <robin.haberkorn@googlemail.com> | 2025-03-19 12:32:10 +0300 |
commit | 257a0bf128e109442dce91c4aaa1d97fed17ad1a (patch) | |
tree | 8880d3e260d6f18e7d9c7935b7886807db510fe3 /src/qreg.c | |
parent | 2b92178287efe3b53237e9d61c69a4bf350c716d (diff) | |
download | sciteco-257a0bf128e109442dce91c4aaa1d97fed17ad1a.tar.gz |
fixed leaking data on rubout
* Objects, that are restored with TECO_DEFINE_UNDO_OBJECT_OWN(),
could actually leak memory on rubout since the old object was not
deleted when overwriting it.
* Now that it is, it is crucial to at least nullify objects/pointers
after calling the corresponding push-function.
These conditions are now explicitly documented.
* The test suite now runs successfully under Valgrind even with full leak checking.
Diffstat (limited to 'src/qreg.c')
-rw-r--r-- | src/qreg.c | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -1708,6 +1708,8 @@ teco_machine_qregspec_auto_complete(teco_machine_qregspec_t *ctx, teco_string_t void teco_machine_qregspec_free(teco_machine_qregspec_t *ctx) { + if (!ctx) + return; teco_machine_stringbuilding_clear(&ctx->machine_stringbuilding); teco_string_clear(&ctx->name); g_free(ctx); |