managing users groups enforce permissions checks.

User needs at least a read permissions on usergroup to be able
to assign it somewhere.

rhodecode/model/repo.py

@@ -41,8 +41,9 b' from rhodecode.model.db import Repositor'
     Statistics, UserGroup, UserGroupRepoToPerm, RhodeCodeUi, RepoGroup,\
     RhodeCodeSetting, RepositoryField
 from rhodecode.lib import helpers as h
-from rhodecode.lib.auth import HasRepoPermissionAny
+from rhodecode.lib.auth import HasRepoPermissionAny, HasUserGroupPermissionAny
 from rhodecode.lib.exceptions import AttachedForksError
+from rhodecode.model.scm import UserGroupList
 
 log = logging.getLogger(__name__)
 
@@ -140,7 +141,9 b' class RepoModel(BaseModel):'
     def get_users_groups_js(self):
         users_groups = self.sa.query(UserGroup)\
             .filter(UserGroup.users_group_active == True).all()
-
+        users_groups = UserGroupList(users_groups, perm_set=['usergroup.read',
+                                                             'usergroup.write',
+                                                             'usergroup.admin'])
         return json.dumps([
             {
              'id': gr.users_group_id,
@@ -472,6 +475,9 b' class RepoModel(BaseModel):'
                     repo=repo, user=member, perm=perm
                 )
             else:
+                #check if we have permissions to alter this usergroup
+                if HasUserGroupPermissionAny('usergroup.read', 'usergroup.write',
+                                             'usergroup.admin')(member):
                 self.grant_users_group_permission(
                     repo=repo, group_name=member, perm=perm
                 )
@@ -482,6 +488,9 b' class RepoModel(BaseModel):'
                     repo=repo, user=member, perm=perm
                 )
             else:
+                #check if we have permissions to alter this usergroup
+                if HasUserGroupPermissionAny('usergroup.read', 'usergroup.write',
+                                             'usergroup.admin')(member):
                 self.grant_users_group_permission(
                     repo=repo, group_name=member, perm=perm
                 )

rhodecode/model/repos_group.py

@@ -169,6 +169,7 b' class ReposGroupModel(BaseModel):'
     def _update_permissions(self, repos_group, perms_new=None,
                             perms_updates=None, recursive=False):
         from rhodecode.model.repo import RepoModel
+        from rhodecode.lib.auth import HasUserGroupPermissionAny
         if not perms_new:
             perms_new = []
         if not perms_updates:
@@ -220,12 +221,18 b' class ReposGroupModel(BaseModel):'
                     _set_perm_user(obj, user=member, perm=perm)
                 ## set for user group
                 else:
+                    #check if we have permissions to alter this usergroup
+                    if HasUserGroupPermissionAny('usergroup.read', 'usergroup.write',
+                                                 'usergroup.admin')(member):
                     _set_perm_group(obj, users_group=member, perm=perm)
             # set new permissions
             for member, perm, member_type in perms_new:
                 if member_type == 'user':
                     _set_perm_user(obj, user=member, perm=perm)
                 else:
+                    #check if we have permissions to alter this usergroup
+                    if HasUserGroupPermissionAny('usergroup.read', 'usergroup.write',
+                                                 'usergroup.admin')(member):
                     _set_perm_group(obj, users_group=member, perm=perm)
             updates.append(obj)
             #if it's not recursive call

rhodecode/model/users_group.py

@@ -63,6 +63,7 b' class UserGroupModel(BaseModel):'
 
     def _update_permissions(self, user_group, perms_new=None,
                             perms_updates=None):
+        from rhodecode.lib.auth import HasUserGroupPermissionAny
         if not perms_new:
             perms_new = []
         if not perms_updates:
@@ -76,6 +77,9 b' class UserGroupModel(BaseModel):'
                     user_group=user_group, user=member, perm=perm
                 )
             else:
+                #check if we have permissions to alter this usergroup
+                if HasUserGroupPermissionAny('usergroup.read', 'usergroup.write',
+                                             'usergroup.admin')(member):
                 self.grant_users_group_permission(
                     target_user_group=user_group, user_group=member, perm=perm
                 )
@@ -86,6 +90,9 b' class UserGroupModel(BaseModel):'
                     user_group=user_group, user=member, perm=perm
                 )
             else:
+                #check if we have permissions to alter this usergroup
+                if HasUserGroupPermissionAny('usergroup.read', 'usergroup.write',
+                                             'usergroup.admin')(member):
                 self.grant_users_group_permission(
                     target_user_group=user_group, user_group=member, perm=perm
                 )