Re: CVS commit: basesrc/dist/ipf

[Klaus Klein <kleink%reziprozitaet.de@localhost>]

Martti Kuparinen <martti%netbsd.org@localhost> 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