On Mon, Feb 01, 2016 at 07:24:39PM +0000, Stuart Henderson wrote:
> On 2016/02/01 15:02, Joerg Jung wrote:
> > What about smtpd, should be similar, no?
> 
> This would do the trick. It loses the getrlimit/setrlimit dance that
> config_process() normally does, but I'm not sure if that is really
> needed for the parent process anyway (mine only has 11 FDs so it's
> not in any danger of running out).

I think that is fine and makes sense to me, ok jung@

But please wait for Gilles to comment.
 
> Index: etc/rc.d/smtpd
> ===================================================================
> RCS file: /cvs/src/etc/rc.d/smtpd,v
> retrieving revision 1.5
> diff -u -p -r1.5 smtpd
> --- etc/rc.d/smtpd    26 Dec 2015 09:55:15 -0000      1.5
> +++ etc/rc.d/smtpd    1 Feb 2016 19:22:33 -0000
> @@ -6,7 +6,6 @@ daemon="/usr/sbin/smtpd"
>  
>  . /etc/rc.d/rc.subr
>  
> -pexp="smtpd: \[priv\]"
>  rc_reload=NO
>  
>  rc_cmd $1
> Index: usr.sbin/smtpd/smtpd.c
> ===================================================================
> RCS file: /cvs/src/usr.sbin/smtpd/smtpd.c,v
> retrieving revision 1.272
> diff -u -p -r1.272 smtpd.c
> --- usr.sbin/smtpd/smtpd.c    27 Jan 2016 12:46:03 -0000      1.272
> +++ usr.sbin/smtpd/smtpd.c    1 Feb 2016 19:22:33 -0000
> @@ -667,8 +667,6 @@ main(int argc, char *argv[])
>  
>       fork_peers();
>  
> -     config_process(PROC_PARENT);
> -
>       imsg_callback = parent_imsg;
>       event_init();
>  
> 

Reply via email to