flock usage

[Time Waster]

Is this a stupid use of flock:

FILE *fp=fopen(SOME_ FILE_CONSTANT," r+");
flock(fileno(fp ),LOCK_EX);
something important here, including reads and a write to fp
flock(fileno(fp ),LOCK_UN);
fclose(fp);

Does this accomplish real locking, or just narrow down the
race quite a bit? (Race existing between the unlock and the
fclose(), i guess.)

Normally I would think you'd want a separate file to do
nothing but the locking, and guard the use of the real data
file with locks on the lockfile.

Also, assuming the above is stupid, is the following a wee
bit smarter:

FILE *fp=fopen(SOME_ FILE_CONSTANT," r+");
flock(fileno(fp ),LOCK_EX);
something important here, including reads and a write to fp
fflush(fp); <<---- at least try to make
fdatasync(fp); <<---- sure contents out before unlock
flock(fileno(fp ),LOCK_UN);
fclose(fp);

(Or does this add very little?)
TIA!

[Keith Thompson]

bf*@fenway.UUCP (Time Waster) writes:

Is this a stupid use of flock:

FILE *fp=fopen(SOME_ FILE_CONSTANT," r+");
flock(fileno(fp ),LOCK_EX);
something important here, including reads and a write to fp
flock(fileno(fp ),LOCK_UN);
fclose(fp);

[snip]

This is not a good place to ask. flock() is not a standard C
function; in fact, standard C provides no facility for locking files.

comp.unix.progr ammer is a better place to ask about this -- but since
flock() isn't defined by the POSIX standard either, they might advise
you to use lockf() instead.

--
Keith Thompson (The_Other_Keit h) ks***@mib.org <http://www.ghoti.net/~kst>
San Diego Supercomputer Center <* <http://users.sdsc.edu/~kst>
"We must do something. This is something. Therefore, we must do this."
-- Antony Jay and Jonathan Lynn, "Yes Minister"

[Walter Roberson]

In article <y9IBi.11154$Eh 5.6838@trndny06 >,
Time Waster <no***@nowhere. comwrote:

>Is this a stupid use of flock:

>FILE *fp=fopen(SOME_ FILE_CONSTANT," r+");
flock(fileno(f p),LOCK_EX);
something important here, including reads and a write to fp
flock(fileno(f p),LOCK_UN);
fclose(fp);

flock() is not defined by the C language; it's properties are
OS specific; for example, there are important differences
between flock() for BSD or System V Unices.

flock() isn't even defined by POSIX.1. If you were intending to
use extensions, you could at least use extensions defined by
the POSIX.1 standard, such as using the POSIX fcntl() with
F_SETLK.

The most obvious stupidity in the code is that it doesn't
check the return values from flock(), so it will go ahead
and scribble on the file if a lock is denied. Not checking
that the open worked is stupid too.
You would probably have better success discussing this in
a newsgroup more specific to the variety of operating system
you are targetting.
--
All is vanity. -- Ecclesiastes

[CBFalconer]

Time Waster wrote:

>
Is this a stupid use of flock:

FILE *fp=fopen(SOME_ FILE_CONSTANT," r+");
flock(fileno(fp ),LOCK_EX);
something important here, including reads and a write to fp
flock(fileno(fp ),LOCK_UN);
fclose(fp);

Maybe you should be asking this somewhere control of herds of birds
is topical. Standard C contains no such routine as 'flock', which
is thus off-topic on c.l.c.

--
Chuck F (cbfalconer at maineline dot net)
Available for consulting/temporary embedded and systems.
<http://cbfalconer.home .att.net>
--
Posted via a free Usenet account from http://www.teranews.com

[pete]

CBFalconer wrote:

>
Time Waster wrote:

Is this a stupid use of flock:

FILE *fp=fopen(SOME_ FILE_CONSTANT," r+");
flock(fileno(fp ),LOCK_EX);
something important here, including reads and a write to fp
flock(fileno(fp ),LOCK_UN);
fclose(fp);

Maybe you should be asking this somewhere control of herds of birds
is topical.

Flocks can contain either feathers or wool.

--
pete