Правильно ли организовал работу flock?

Question

[Bur Ov]

Правильно ли организовал защиту от одновременного запуска php скрипта по cron?

$lock = sys_get_temp_dir()."/test.lock";
    $aborted = file_exists($lock) ? filemtime($lock) : null;
    $fp = fopen($lock, 'w');

    if (!flock($fp, LOCK_EX|LOCK_NB)) {
         exit( 'already running' );
    }

 // Тут код скрипта

    // снятие блокировки по окончанию работы 
    // если этот callback, не будет выполнен, то блокировка 
    // все равно будет снята ядром, но файл останется
    register_shutdown_function(function() use ($fp, $lock) {
        flock($fp, LOCK_UN);
        fclose($fp);
        unlink($lock);
    });

Если я теперь поставлю скрипт test.php на cron, то если скрипт еще работает, то крон его не выполнит, так?

Answer 1

[Григорий Есин]

Вообще да, правильно.
Но наличие lock-файла может говорить не только о том, что скрипт был "убит" но и потому, что он сейчас работает.
Рекомендую сохранить pid в этот файл и проверять, что процесс с этим pid существует.

Comments

[Bur Ov]

ак эта проверка то и делает, if (!flock($fp, LOCK_EX|LOCK_NB)) {
exit( 'already running' );
}
если не удается создать лок файл, то скрипт выполняется, если удается создать - то выполняется скрипт

[Bur Ov]

*то выполняется сценарий дальше

[Bur Ov]

Если скрипт зависнет, либо его закроют из вне, то register_shutdown_function снимет лок же

[Григорий Есин]

При условии, что скрипт не будет убит сигналом sigkill: может придти oomkiller.

[Григорий Есин]

Попробуйте убить скрипт командой kill -9 (pid скрипта)