--- a/samba/filepropertiesplugin/authhelper.cpp +++ b/samba/filepropertiesplugin/authhelper.cpp @@ -88,7 +88,7 @@ ActionReply AuthHelper::addtogroup(const // TODO: add ability to resolve remote UID via KAuth and verify the request (or even reduce the arguments down to // only the group and resolve the UID) // Keep this condition in sync with the one in groupmanager.cpp - if (!group.contains(QLatin1String("samba")) || group.contains(QLatin1String("admin")) || + if (!group.contains(QLatin1String("share")) || group.contains(QLatin1String("admin")) || group.contains(QLatin1String("root"))) { return ActionReply::HelperErrorReply(); } --- a/samba/filepropertiesplugin/groupmanager.cpp +++ b/samba/filepropertiesplugin/groupmanager.cpp @@ -55,7 +55,7 @@ GroupManager::GroupManager(QObject *pare // Now see if the group is set to something valid // Keep this condition in sync with the one in authhelper.cpp - else if (!m_targetGroup.contains(QLatin1String("samba")) + else if (!m_targetGroup.contains(QLatin1String("share")) || m_targetGroup.contains(QLatin1String("admin")) || m_targetGroup.contains(QLatin1String("root")) ) {