Can anyone advise on this one?

DBMan's auth.pl file has a number of open file and close file statements, but only some are covered by flock (file locking).

In general, is it good practice to flock all open and close statements or is there some other reason for not file locking?

Thanks in advance for any comments!