Question

I see people using flock like this:

if (!$fp = fopen($file_name, 'wb'))  
{  
    return FALSE;  
}  

if (flock($fp, LOCK_EX))  
{  
    fwrite($fp, serialize($data));  
    flock($fp, LOCK_UN);  
}

Also this:

if (!$fp = @fopen($file_name, 'rb'))  
{  
    return FALSE;  
}  

flock($fp, LOCK_SH);  

$data = '';  

if (filesize($file_name) > 0)  
{  
    $data = unserialize(fread($fp, filesize($file_name)));  
}  

But isn’t there a chance that someone else will edit the file between the fopen call and the flock call? and the same question for fread

---

EDIT:
To clarify why I’m asking this… I’m basing my question on the code here, In a mysql caching situation, what’s to stop 20 people from all being able to access the file at the same time if they all can get in between the fopen and flock?

Is that code foolproof?

Answer 1

You ask:

isn’t there a chance that someone else will edit the file between the fopen call and the flock call? and the same question for fread

Yes, no, maybe. Short answer: assume “yes” and act carefully.

Yes, in that traditional flock()-based locking is merely advisory, so other processes (or even the same process) are free to disregard the locks. In practice, this is not a problem, as flock() is used by well-behaved client code — you don’t read until you get a LOCK_SH, and you don’t write unless you’ve obtained a LOCK_EX — on application-specific files.

No, in that PHP’s implementation of flock() may be mandatory on certain operating systems, per the documentation, which might also require support from the filesystem (e.g., as with the mand option under Linux). So, other processes could not disregard those locks.

Maybe, in that the streams subsystem in PHP 5 implements some locking bookkeeping beyond that provided by the operating system. This may, for example, prevent the same process (but not another) from disregarding its own otherwise advisory locks. The behavior might surprise some. Even so, this kind of locking would not be mandatory between unrelated processes.

For portability, just assume the weakest semantics (the “yes” above) and restrict flock()ing to well-behaved code on application-specific lockfiles chosen in advance.