[Fwd: Serious problem with mount(8)]
des at des.no
Wed May 23 07:30:15 UTC 2007
Joe Marcus Clarke <marcus at FreeBSD.org> writes:
> Dag-Erling Smørgrav <des at des.no> writes:
> > Better yet, extend the pidfile API with a function which reads the
> > contents of a PID file and also checks whether it's locked.
> I'd be happy to do this. Is my approach with this code sound (i.e. can
> I simply port this to pidfile(3)), or should I take another approach?
You expose yourself to all sorts of race conditions by opening the file
twice... what you should do is something like (off the top of my head):
fd = open(pidfile, O_RDONLY);
fcntl(fd, F_GETLK, &fl)
if (fl.l_type == F_UNLCK)
read(fd, buf, fsb.st_len);
if (atoi(buf) != fl.l_pid)
if (sb.st_dev != fsb.st_dev || sb.st_ino != fsb.st_ino)
with appropriate error checking, of course.
Dag-Erling Smørgrav - des at des.no
More information about the freebsd-current