search

Ultimate-Hosts-Blacklist / Ultimate.Hosts.Blacklist

The Ultimate Unified Hosts file for protecting your network, computer, smartphones and Wi-Fi devices against millions of bad web sites. Protect your children and family from gaining access to bad web sites and protect your devices and pc from being infected with Malware or Ransomware.
MIT License
1.24k stars 155 forks source link

Accounts for a root-user with a different name #606

Closed ngaro closed 3 years ago

ngaro commented 3 years ago

Purely in theory, the root user can have another name but he will always have the UID 0. Although I have never seen this is in real life, this PR solve this 'problem".

It's also a couple of milliseconds faster because id is not forced to look up a name. (Even when asking for a name it will always look up the UID first)

spirillen commented 3 years ago

good idea :+1:

ngaro commented 3 years ago

An extra commit because of rustysnake's excellent idea in #605 . I know it's "not done" to do multiple things in 1 pr, but i'm planning some other pr's and i don't want too many merge conflicts in case some pr's are merged and some are closed.

These are extremely readable commits anyway...

ngaro commented 3 years ago

This is 'the simple version', i would suggest using #608 where i also do a lot more (but everything is still strongly related)

funilrys commented 3 years ago

Closing as the change is into the more complete #608.