Author Topic: <Resolved> GetVirtualBox adding unwanted members to vboxusers group  (Read 332 times)

Offline kjpetrie

  • PCLinuxOS Tester
  • Hero Member
  • *******
  • Posts: 3996
I suspect it's always done it, but it has now become an issue to me:

GetVirtualBox adds all users who can log in to a desktop to the vboxusers group, whether the system administrator thinks they should be there or not. This means every time I upgrade I get unwanted users added to the group and I have to remember to remove them again.

An upgrade script should not interfere with administrative decisions. Could the script be modified so it only creates the group and adds users if the group does not already exist?

Alternatively, perhaps it shouldn't populate the group at all and just instruct the user to add those users required to the group.
« Last Edit: October 04, 2012, 04:50:37 AM by kjpetrie »
-----------
KJP
-----------------------------------------------------------
PClos64 RC1 on Intel D945GCLF2 motherboard (Atom 330), 2GB DDR2 RAM, Maxtor STM325031, HL-DT-ST DVDRAM GSA-H42N, Amilo LSL 3220T monitor. Also Acer 5810TG (with custom kernel) and Asus eeePC 2G surf

Offline Just17

  • PCLinuxOS Tester
  • Super Villain
  • *******
  • Posts: 10668
  • MLUs Forever!
Re: GetVirtualBox adding unwanted members to vboxusers group
« Reply #1 on: October 03, 2012, 04:57:38 PM »
I must watch out for this ......  it seems I do not have this here, but am not certain.

I have 3 users and only one is a member of vboxusers ......  but one account was likely created after i updated vbox last .......  but the other user is a long established one .......

thanks for raising it ....  will keep a look-out when I next update vbox.

MLUs rule the roost!

Linux XPS 3.4.38-pclos1.bfs  64 bit
Intel Core2 Quad CPU Q9450 @ 2.66GHz
4 GB RAM
MCP51 High Def Audio
GeForce GTX 550 Ti
PHILIPS  ‎DVD+-RW DVD8701
‎Logitech ‎BT Mini-Receiver
Afatech DTT

Offline pinoc

  • Global Moderator
  • Hero Member
  • *****
  • Posts: 2847
    • other projects...
Re: GetVirtualBox adding unwanted members to vboxusers group
« Reply #2 on: October 04, 2012, 01:25:03 AM »
GetVirtualBox adds all users who can log in to a desktop to the vboxusers group.

Yes, that makes perfect sense, has always been like that, and was setup this way by Tex. I don't see any reason why this should be changed as this is required to allow users to actually use VirtualBox. I will definitely not change this to instruct the user to add those users required to the group as such a move would be completely user-unfriendly and un-needed by 99.9 % of all users.
If you need to exclude some specific users from using VirtualBox then this is a specific need for you but not for the majority of all users, unless I completely misunderstand your point of course.
-p.

Offline kjpetrie

  • PCLinuxOS Tester
  • Hero Member
  • *******
  • Posts: 3996
Re: GetVirtualBox adding unwanted members to vboxusers group
« Reply #3 on: October 04, 2012, 03:35:46 AM »
I think you do misunderstand. My point is that upgrading an application should not change existing users' permissions or other configuration matters.

These are set by the user of the machine and should remain as set.

What the script does is reasonable for first installation, but not for an upgrade, where the user would expect the set-up to remain.

I have created a user for a special purpose which is exposed to the Internet, so I want to lock that user down, and I don't want my work undone at every upgrade.

What I think is needed is a test of whether vboxusers already exists, and if it does, just leave it as it is.
-----------
KJP
-----------------------------------------------------------
PClos64 RC1 on Intel D945GCLF2 motherboard (Atom 330), 2GB DDR2 RAM, Maxtor STM325031, HL-DT-ST DVDRAM GSA-H42N, Amilo LSL 3220T monitor. Also Acer 5810TG (with custom kernel) and Asus eeePC 2G surf

Offline pinoc

  • Global Moderator
  • Hero Member
  • *****
  • Posts: 2847
    • other projects...
Re: GetVirtualBox adding unwanted members to vboxusers group
« Reply #4 on: October 04, 2012, 03:46:20 AM »
What I think is needed is a test of whether vboxusers already exists, and if it does, just leave it as it is.
ok, done for the next update of getvirtualbox.
-p.

Offline kjpetrie

  • PCLinuxOS Tester
  • Hero Member
  • *******
  • Posts: 3996
Re: GetVirtualBox adding unwanted members to vboxusers group
« Reply #5 on: October 04, 2012, 04:49:54 AM »
Thank you.
-----------
KJP
-----------------------------------------------------------
PClos64 RC1 on Intel D945GCLF2 motherboard (Atom 330), 2GB DDR2 RAM, Maxtor STM325031, HL-DT-ST DVDRAM GSA-H42N, Amilo LSL 3220T monitor. Also Acer 5810TG (with custom kernel) and Asus eeePC 2G surf