Always check the return value from things such as open and flock. And it'd be better to use the symbolic constants from Fcntl rather than hardcoding `3' (assuming traditional values that would seem to be LOCK_SH|LOCK_EX, which doesn't make much sense).