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