bb-qq / r8152

Synology DSM driver for Realtek RTL8152/RTL8153/RTL8156 based adapters
GNU General Public License v2.0
2.07k stars 184 forks source link

Why need spk_su.c? #137

Closed riverscn closed 3 years ago

riverscn commented 3 years ago

I read https://help.synology.com/developer-guide/privilege/privilege_config.html and have a question.

Why not write conf/privilege file in the repo, instead of spk_su.c during installation process?

bb-qq commented 3 years ago

Since DSM 7.0, root privilege is only allowed for packaged created by Synology. https://help.synology.com/developer-guide/privilege/preface.html

To avoid this limitation, we need to modify conf/privilege after the timing when the Package Manager checks its value. And privilege escalation is required to modify conf/privilege because post-process of installation is executed as logon user of DSM.

riverscn commented 3 years ago

TIL!

So it seems there is no official configuration for user package which contains a hardware driver.

bb-qq commented 3 years ago

Yes, you are right.