On Thu, Apr 2, 2026 at 7:44 PM Pádraig Brady <[email protected]> wrote: > Thank you for your continued work on this.
I've exercised the patchstack with ASAN and there is a small update: the attached patch adds free() before exit() to make the tests pass. Is there anything else I can do to QA it? E.g. is there a way for me to trigger CI build for the branch to check for regressions? -- WBRBW, Leonid Evdokimov, https://darkk.net.ru tel:+79816800702 PGP: 6691 DE6B 4CCD C1C1 76A0 0D4A E1F2 A980 7F50 FAB2
From e6d469e68945f58f06207553b69490a8f824b840 Mon Sep 17 00:00:00 2001 From: Leonid Evdokimov <[email protected]> Date: Fri, 10 Apr 2026 00:36:39 +0300 Subject: [PATCH 8/8] split: make tests pass with AddressSanitizer free()'ing the memory right before main_exit() is one way to silence ASAN complaining about memory leak. --- src/split.c | 1 + 1 file changed, 1 insertion(+) diff --git src/split.c src/split.c index aa5cf42b0..3a66ac1e6 100644 --- src/split.c +++ src/split.c @@ -1151,6 +1151,7 @@ bytes_cdc_split (enum Cdc_type const hash, intmax_t const avgsz, } if (n_read < 0) error (EXIT_FAILURE, errno, "%s", quotef (infile)); + free ((void *)terminator); } /* Split into pieces of exactly N_LINES lines. -- 2.34.1
