From 91e66c481f31859ce04c1214d2ed8b264bd547f3 Mon Sep 17 00:00:00 2001 From: "Todd C. Miller" Date: Tue, 13 Aug 2013 14:53:55 -0600 Subject: [PATCH] Fix setting of mailer argv[0] to basename of mailerpath. No need to strdup() mailerpath as it is not modified. --- plugins/sudoers/logging.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/plugins/sudoers/logging.c b/plugins/sudoers/logging.c index a53581d94..2b4c48b7c 100644 --- a/plugins/sudoers/logging.c +++ b/plugins/sudoers/logging.c @@ -640,7 +640,7 @@ send_mail(const char *fmt, ...) case 0: { char *argv[MAX_MAILFLAGS + 1]; - char *mpath, *mflags; + char *mflags, *mpath = def_mailerpath; int i; /* Child, set stdin to output side of the pipe */ @@ -657,8 +657,7 @@ send_mail(const char *fmt, ...) /* Build up an argv based on the mailer path and flags */ mflags = estrdup(def_mailerflags); - mpath = estrdup(def_mailerpath); - if ((argv[0] = strrchr(mpath, ' '))) + if ((argv[0] = strrchr(mpath, '/'))) argv[0]++; else argv[0] = mpath;