doc-en icon indicating copy to clipboard operation
doc-en copied to clipboard

sem_get add warning about freeing variable

Open cjhewett opened this issue 1 year ago • 1 comments

From manual page: https://php.net/function.sem-get

sem_get() has an important gotcha where if the variable is freed then the semaphore is released. I propose this is not quite a bug but could do with a red warning box e.g.

Warning: sem_get will also auto_release the semaphore when the variable which holds it is freed (rather than the process ending). Ensure you store the result in a variable which is not overwritten, unset, or goes out of scope while the semaphore should be alive.

This behaviour can be demonstrated with the following:

$semGet = sem_get(1);

var_dump(sem_acquire($semGet, true));
// true: semaphore(1) acquired

var_dump(sem_acquire(sem_get(1), true));
// false: semaphore(1) already acquired

unset($semGet);

var_dump(sem_acquire(sem_get(1), true));
// true: semaphore(1) released by variable unset() so can be acquired again

cjhewett avatar Jul 25 '24 13:07 cjhewett

I would presume that this changed when the resources were converted to objects; might make sense to add that info to the note (or maybe to the changelog entry).

cmb69 avatar Jul 26 '24 18:07 cmb69