View Issue Details

This issue affects 1 person(s).
 256
IDProjectCategoryView StatusLast Update
11063Bug reportsSecuritypublic2016-04-28 08:59
Reporterfvanderstarre Assigned Toc_schmitz  
PrioritynoneSeverityminor 
Status closedResolutionno change required 
Product Version2.06+ 
Summary11063: Superadmin account can change admin password
Description

When logged in with a superadmin account, I noticed I could change the (main) admin account's password. I think this is potentially unsafe, but is it intentional?

Steps To Reproduce

Log in with superadmin account - go to manage Survey Administrators - select the "admin" account - password is updateable (and can be saved).

TagsNo tags attached.
Attached Files
admin.jpg (38,183 bytes)   
admin.jpg (38,183 bytes)   
Bug heat256
Complete LimeSurvey version number (& build)2.06+ build 160129
I will donate to the project if issue is resolvedNo
BrowserAny
Database type & versionMS SQL server 2008R2 10.50.6220 (X64)
Server OS (if known)DB server: Windows NT 6.1 <X64> build 7601 SP1
Webserver software & version (if known)Linux RHEL 7, Apache 2.4.6
PHP Version5.6.19
Git branch / Pull Request URL

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2016-04-28 08:17

developer   ~37919

Last edited: 2016-04-28 08:17

It's needed. And it's like this on all major CMS (Drupal/Wordpress/Spip etc ....)

If you don't like it : think we can use Permission event to fix it (in Plugin)

fvanderstarre

fvanderstarre

2016-04-28 08:23

reporter   ~37923

Do you mean it's necessary, in case the admin's password is lost (that's what happened to me)?

DenisChenu

DenisChenu

2016-04-28 08:54

developer   ~37924

It's not necessary, but it's really clean (for me).

Sometimes : user can be "false" user for API for example too : necessary

c_schmitz

c_schmitz

2016-04-28 08:59

administrator   ~37927

Well, you should not make someone superadmin you don't trust the superadmin permission. ANy superadmin can export the whole DB including password hashes - a simple password change pales compared to that.

Issue History

Date Modified Username Field Change
2016-04-28 07:58 fvanderstarre New Issue
2016-04-28 07:58 fvanderstarre File Added: admin.jpg
2016-04-28 08:17 DenisChenu Note Added: 37919
2016-04-28 08:17 DenisChenu Note Edited: 37919
2016-04-28 08:23 fvanderstarre Note Added: 37923
2016-04-28 08:54 DenisChenu Note Added: 37924
2016-04-28 08:59 c_schmitz Note Added: 37927
2016-04-28 08:59 c_schmitz Status new => closed
2016-04-28 08:59 c_schmitz Assigned To => c_schmitz
2016-04-28 08:59 c_schmitz Resolution open => no change required