This breaks things due to restarting the service multiple times once here and more times when custom_php_resync_config_command is run on install, which in turn runs freeradius_settings_resync() which restarts the service, and then freeradius_users_resync() runs which restarts the service yet again.

It will restart after freeradius_users_resync() is finished.

…ectory

Probably some leftover from 2.2.x/PBI junk.

If $frlib was empty, the code would run chown -R on the entire /usr/local/lib. Create a function with sanity checks, use PHP's chown()/chgrp() functions instead of exec() and refuse to recursively chown invalid targets.

…resync()

Needed to avoid more restarts madness