Subject: Re: CVS commit: basesrc/dist/ipf
To: None <martti@netbsd.org>
From: Klaus Klein <kleink@reziprozitaet.de>
List: source-changes
Date: 09/19/2002 10:54:20
Martti Kuparinen <martti@netbsd.org> writes:

> Module Name:	basesrc
> Committed By:	martti
> Date:		Thu Sep 19 08:11:15 UTC 2002
> 
> Modified Files:
> 	basesrc/dist/ipf: ipmon.c
> 	basesrc/dist/ipf/man: ipnat.5
> 
> Log Message:
> Resync with official IPF

Is there any particular reason for making this change to ipmon.c,
effectively backing out rev. 1.10? For a "tight" integration the
pidfile(3) interface should be used, but not doing so doesn't justify
a pidfile format different from that used by other daemons in the base
system (well, with the notable exception of sendmail) either.

--- ipmon.c     2002/09/19 08:08:18     1.14
+++ ipmon.c     2002/09/19 08:11:13     1.15
[...]
@@ -1187,7 +1183,7 @@
                fprintf(stderr, "unable to open/create pid file: %s\n", file);
                return;
        }
-       fprintf(fp, "%d\n", getpid());
+       fprintf(fp, "%d", getpid());
        fclose(fp);
        close(fd);
 }



- Klaus