SHARYANTO::File::Flock unlink-related race conditions

[andersk]

In SHARYANTO::File::Flock, your redo TRY logic is written incorrectly. If $path has been unlinked by another process, then @st2 will become empty, but your code fails to redo TRY in that case and incorrectly returns success.

Additionally, you have to be more careful never to unlink the lockfile while the current process doesn’t have it locked. That may happen in your code if release or unlock is called before the object is destroyed.

[perlancar]

Thanks, I've handled the case when @st2 is empty. commit:be6ae65

Also, file is now unlinked only when current process is holding the lock. commit:6bd9753