View Issue Details

Related ChangesetsJump to NotesJump to History
This issue affects 1 person(s).
 2
IDProjectCategoryView StatusDate SubmittedLast Update
09569Bug reportsOtherpublic2015-03-18 00:322015-05-20 11:11
ReporterchrisSL Assigned Toc_schmitz  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.05+ 
Fixed in Version2.05+ 
Summary09569: admin with uid>1 cannot set superadmin permissions
Description

If the user id of the admin user isn't "1", the admin looses his special permissions, e.g. to grant superadmin rights to other users.

Steps To Reproduce

For database setups running a master-master configuration there are special recommendations. For example to set

auto_increment_increment =
auto_increment_offset =

to special values to avoid conflicts between the nodes.

If a server is set do

auto_increment_increment = 2
auto_increment_offset = 2

it will always set auto_increment columns to 2, 4, 6, 8, ...

Installing limesurvey in such an environment will produce an admin with uid=2.

This was true at least for limesurvey-1.91plus-build120226 and has not been taken in account during the upgrade procedure.

Additional Information

I would recommend to not hardcode the uid=1. Either the uid should be configurable or the correct value should be taken from the database min(uid). Maybe non-standard setups with admin uid>1 should be fixed during upgrade.

Changing the loginID values in
application/models/Permission.php
application/controllers/admin/useraction.php

temporarily solved the issue for me.

TagsNo tags attached.
Bug heat2
Complete LimeSurvey version number (& build)build150310
I will donate to the project if issue is resolvedNo
Browser
Database type & versionmysql
Server OS (if known)SLES 11
Webserver software & version (if known)Apache 2
PHP Version5.3.17
Git branch / Pull Request URL

Users monitoring this issue

There are no users monitoring this issue.

Activities

c_schmitz

c_schmitz

2015-05-13 12:34

administrator   ~32201

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=15179
c_schmitz

c_schmitz

2015-05-13 12:36

administrator   ~32202

Fix committed to 2.06 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=15180
c_schmitz

c_schmitz

2015-05-13 12:36

administrator   ~32203

Last edited: 2015-05-13 12:36

On admin user creation during installation the ID is now fixed to 1 for MySQL
c_schmitz

c_schmitz

2015-05-20 11:11

administrator   ~32241

Version 2.05+ Build 150520 released.

Related Changesets

LimeSurvey: master 2b5f409a

2015-05-13 12:34

c_schmitz


Details Diff		 Fixed issue 09569: Admin with uid>1 cannot set superadmin permissions Affected Issues
09569
mod - application/controllers/InstallerController.php Diff File

LimeSurvey: 2.06 285c2951

2015-05-13 12:36

c_schmitz


Details Diff		 Fixed issue 09569: Admin with uid>1 cannot set superadmin permissions Affected Issues
09569
mod - application/controllers/InstallerController.php Diff File

Issue History

Date Modified Username Field Change
2015-03-18 00:32 chrisSL New Issue
2015-03-18 09:24 Mazi Assigned To => c_schmitz
2015-03-18 09:24 Mazi Status new => assigned
2015-05-13 12:34 c_schmitz Changeset attached => LimeSurvey master 2b5f409a
2015-05-13 12:34 c_schmitz Note Added: 32201
2015-05-13 12:34 c_schmitz Resolution open => fixed
2015-05-13 12:36 c_schmitz Changeset attached => LimeSurvey 2.06 285c2951
2015-05-13 12:36 c_schmitz Note Added: 32202
2015-05-13 12:36 c_schmitz Note Added: 32203
2015-05-13 12:36 c_schmitz Status assigned => resolved
2015-05-13 12:36 c_schmitz Fixed in Version => 2.05+
2015-05-13 12:36 c_schmitz Note Edited: 32203
2015-05-20 11:11 c_schmitz Note Added: 32241
2015-05-20 11:11 c_schmitz Status resolved => closed