aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/qreg.c
diff options
context:
space:
mode:
authorRobin Haberkorn <robin.haberkorn@googlemail.com>2025-03-19 12:32:10 +0300
committerRobin Haberkorn <robin.haberkorn@googlemail.com>2025-03-19 12:32:10 +0300
commit257a0bf128e109442dce91c4aaa1d97fed17ad1a (patch)
tree8880d3e260d6f18e7d9c7935b7886807db510fe3 /src/qreg.c
parent2b92178287efe3b53237e9d61c69a4bf350c716d (diff)
downloadsciteco-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.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/qreg.c b/src/qreg.c
index 4beea74..1b97689 100644
--- a/src/qreg.c
+++ b/src/qreg.c
@@ -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);