prepstdreply: fix resource leaks in error paths

Use close_text() instead of free() when cleaning up txt in error paths.
The txt parameter is a fully initialized text structure from open_text(),
so it needs proper cleanup via close_text() to free all internal resources.

Also add missing close_text() call when prepstdreply_to() fails.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

diff --git a/src/prepstdreply.c b/src/prepstdreply.c
index 0ff338b300a9c4050cc98df64996a021b96ffc9f..7ef77ffa3dcd766647c0d70635a9c3dfce101179 100644 (file)
--- a/src/prepstdreply.c
+++ b/src/prepstdreply.c
@@ -1778,13 +1778,14 @@ char *prepstdreply(text *txt, struct ml *ml, const char *from, const char *to, c
        if(outfd < 0) {
                log_error(LOG_ARGS, "Could not open std mail %s", retstr);
                free(retstr);
-               free(txt);
+               close_text(txt);
                return NULL;
        }
        if (!prepstdreply_to(txt, ml, from, to, replyto, outfd, NULL)) {
                close(outfd);
                unlink(retstr);
                free(retstr);
+               close_text(txt);
                return (NULL);
        }
        return (retstr);