From keramida@ceid.upatras.gr  Wed Aug 22 21:24:44 2001
Return-Path: <keramida@ceid.upatras.gr>
Received: from mailsrv.otenet.gr (mailsrv.otenet.gr [195.170.0.5])
	by hub.freebsd.org (Postfix) with ESMTP id CA45037B40A
	for <FreeBSD-gnats-submit@freebsd.org>; Wed, 22 Aug 2001 21:24:43 -0700 (PDT)
	(envelope-from keramida@ceid.upatras.gr)
Received: from hades.hell.gr (patr530-b035.otenet.gr [195.167.121.163])
	by mailsrv.otenet.gr (8.11.1/8.11.1) with ESMTP id f7N4OdM01167;
	Thu, 23 Aug 2001 07:24:39 +0300 (EEST)
Received: (from charon@localhost)
	by hades.hell.gr (8.11.4/8.11.4) id f7N4OtJ02068;
	Thu, 23 Aug 2001 07:24:55 +0300 (EEST)
	(envelope-from keramida@ceid.upatras.gr)
Message-Id: <20010823072455.A1526@hades.hell.gr>
Date: Thu, 23 Aug 2001 07:24:55 +0300
From: Giorgos Keramidas <keramida@ceid.upatras.gr>
To: peter@dataloss.net
Cc: FreeBSD-gnats-submit@freebsd.org
In-Reply-To: <20010220010838.1196.qmail@massive.dataloss.net>; from peter@dataloss.net on Tue, Feb 20, 2001 at 01:08:38AM -0000
Subject: Re: mailwrapper invokes sendmail when resources are tight
References: <20010220010838.1196.qmail@massive.dataloss.net>

>Number:         29972
>Category:       bin
>Synopsis:       Re: mailwrapper invokes sendmail when resources are tight
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    gnats-admin
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Wed Aug 22 21:30:00 PDT 2001
>Closed-Date:    Tue Aug 28 05:21:32 PDT 2001
>Last-Modified:  Wed Oct 26 05:47:47 GMT 2005
>Originator:     
>Release:        
>Organization:
>Environment:
>Description:
 From: Peter Pentchev <roam@orbitel.bg>
 Subject: Re: misc/25218: mailwrapper invokes sendmail when resources are tight
 Date: Fri, 6 Apr 2001 15:20:46 +0300
 
 > Mmmm here's an updated patch, which also removes the now-unused
 > default MTA define from pathnames.h (hint given by the submitter
 > in private e-mail).
 
 This patch failed to apply to my -CURRENT sources CVSup'ed today.
 I have made and tested today a new version of it.
 
 But, instead of chasing mailwrapper's sources in an endless
 patch-update-patch loop, I think we should have some comments on the
 suggestion of this PR from people in -CURRENT.
 
 The question still remains.  Do we *want* mailwrapper to invoke
 /usr/libexec/sendmail/sendmail if it fails to open mailer.conf?  If
 yes, this PR should be closed.  If no, the patch be tested by some
 more people and eventually be committed?
 
 					-giorgos
 
 
 ---- patch begins here --------------------------------------------
 
 Index: mailwrapper.c
 ===================================================================
 RCS file: /home/ncvs/src/usr.sbin/mailwrapper/mailwrapper.c,v
 retrieving revision 1.7
 diff -u -r1.7 mailwrapper.c
 --- mailwrapper.c	2001/08/09 06:20:18	1.7
 +++ mailwrapper.c	2001/08/23 04:01:49
 @@ -123,14 +123,11 @@
  		addarg(&al, argv[len], 0);
  
  	if ((config = fopen(_PATH_MAILERCONF, "r")) == NULL) {
 -		addarg(&al, NULL, 0);
  		openlog("mailwrapper", LOG_PID, LOG_MAIL);
 -		syslog(LOG_INFO, "can't open %s, using %s as default MTA",
 -		    _PATH_MAILERCONF, _PATH_DEFAULTMTA);
 +		syslog(LOG_INFO, "can't open %s", _PATH_MAILERCONF);
  		closelog();
 -		execve(_PATH_DEFAULTMTA, al.argv, envp);
  		freearg(&al, 0);
 -		err(1, "execing %s", _PATH_DEFAULTMTA);
 +		err(1, "opening %s", _PATH_MAILERCONF);
  		/*NOTREACHED*/
  	}
  
 Index: pathnames.h
 ===================================================================
 RCS file: /home/ncvs/src/usr.sbin/mailwrapper/pathnames.h,v
 retrieving revision 1.3
 diff -u -r1.3 pathnames.h
 --- pathnames.h	2000/01/10 03:20:13	1.3
 +++ pathnames.h	2001/08/23 03:56:34
 @@ -32,4 +32,3 @@
   */
  
  #define _PATH_MAILERCONF	"/etc/mail/mailer.conf"
 -#define _PATH_DEFAULTMTA	"/usr/libexec/sendmail/sendmail"
>How-To-Repeat:
>Fix:
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->closed 
State-Changed-By: dd 
State-Changed-When: Tue Aug 28 05:21:32 PDT 2001 
State-Changed-Why:  
not a new pr 

http://www.FreeBSD.org/cgi/query-pr.cgi?pr=29972 
>Unformatted:
