You can't free a string and then use it as the error message to

errx. Instead use warn to print the string, then cleanup and exit
normally.

This fixes a core dump if the executable to be run doesn't exist.

1 files changed, 2 insertions, 1 deletions

diff --git a/usr.sbin/mailwrapper/mailwrapper.c b/usr.sbin/mailwrapper/mailwrapper.c
index 509496b..7d6ad14 100644
--- a/usr.sbin/mailwrapper/mailwrapper.c
+++ b/usr.sbin/mailwrapper/mailwrapper.c
@@ -176,8 +176,9 @@ main(argc, argv, envp)
 	addarg(&al, NULL, 0);
 	execve(to, al.argv, envp);
 	freearg(&al, 0);
+	warn("execing %s", to);
 	free(line);
-	err(1, "execing %s", to);
+	exit(1);
 	/*NOTREACHED*/
 parse_error:
 	freearg(&al, 0);