Skip to content

Commit 57e08e6

Browse files
fix(discover): address QA round 2 findings
- Defensively free p->userconfig in cbm_pipeline_free in case run() was never called (e.g. pipeline created but not executed before free) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 1bfe473 commit 57e08e6

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

src/pipeline/pipeline.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,12 @@ void cbm_pipeline_free(cbm_pipeline_t *p) {
101101
free(p->db_path);
102102
free(p->project_name);
103103
/* gbuf, store, registry freed during/after run */
104+
/* Defensively free userconfig in case run() was never called or panicked */
105+
if (p->userconfig) {
106+
cbm_set_user_lang_config(NULL);
107+
cbm_userconfig_free(p->userconfig);
108+
p->userconfig = NULL;
109+
}
104110
free(p);
105111
}
106112

0 commit comments

Comments
 (0)