NoCatSplash sync patch

"J. Kyllo" <[email protected]>
Newsgroups gmane.network.nocat
Message-ID <[email protected]>
Hello NoCat list.  I have been using NoCatAuth for a while now and was
looking into NoCatSplash.  I run debian so I was looking to build a .deb
for NoCatSplash.  While trying to use the sample rc/init script in the
0.93pre2 tarball, I found that splashd does not sync the pid file.  The
result of this is that the init script is not able to kill the process.

I did some poking around and saw that the pidfile doesn't get closed until
splashd is exiting (and the next action is to unlink it).  Attached is a
one line patch that uses a call to fseek to effectly sync the file.  I
used fseek to keep with the use of FILE* as opposed to fd.

Any thoughts on this?  On one hand, I would think that the pidfile should
really just be closed after it is written.  But I can also understand that
it should stay open with the running daemon.  Either way, this patch
sync's the file and lets the init script work correctly.

Thanks,
Jeff K.

_______________________________________________
NoCat mailing list
[email protected]
http://lists.nocat.net/mailman/listinfo/nocat
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.