Skip to main content

Insecure functionality - User management

Description​

It is possible to assign non-users as treatment managers of vulnerabilities, this will send an email with the information of the vulnerabilities even if the email is not registered on integrates. Even if the user has been removed from all the projects of an organization, the information of the vulnerabilities over the time and the policies is still accessible by the email.

Impact​

  • Grant the ability for unauthorized administrators to manage user access to different system resources.
  • Disclose sensitive information.

Recommendation​

Grant access to the projects only to users that have consented the reception of information from the application. Disable all users from the organizations if they should not have access to any project. Verify that only authorized users have access to information and actions on the application.

Threat​

Authorized external attacker with privileges to add users.

Expected Remediation Time​

⌚ 60 minutes.

Score​

Default score using CVSS 3.1. It may change depending on the context of the vulnerability.

Base​

  • Attack vector: N
  • Attack complexity: L
  • Privileges required: H
  • User interaction: N
  • Scope: U
  • Confidentiality: L
  • Integrity: L
  • Availability: N

Temporal​

  • Exploit code madurity: X
  • Remediation level: X
  • Report confidence: X

Result​

  • Vector string: CVSS:3.1/AV:N/AC:L/PR:H/UI:N/S:U/C:L/I:L/A:N/E:X/RL:X/RC:X
  • Score:
    • Base: 3.8
    • Temporal: 3.8
  • Severity:
    • Base: Low
    • Temporal: Low

Code Examples​

Compliant code​

The application securely checks user authorization credentials before accesing sensitive information

const grantAccess = (req, res) => {
  const user = req.body.user;
  const vulnGroup = req.body.vulnGroup;
  const isVulnInGroup = user.vulnGroups.find(vulnGroup)
  //Only allow user to access the vulnerability if he has the relevant permissions
  if (isValidUser(req.body.user) && isVulnInGroup) {
    if (validateToken == req.body.token){
      assignVulnToUser(req.body.user, vuln);
      //Code to alert user of the reassign vulnerability
    }
  }
}

Non compliant code​

The application does not properly check user permissions before accesing sensitive information

const grantAccess = (req, res) => {
  if (isValidUser(req.body.user)) {
    if (validateToken == req.body.token){
      const vuln = req.body.vulnId;
      assignVulnToUser(req.body.user, vuln);
      //Code to alert user of the reassign vulnerability
    }
  }
}

Requirements​

free trial

Search for vulnerabilities in your apps for free with our automated security testing! Start your 21-day free trial and discover the benefits of our Continuous Hacking Machine Plan. If you prefer a full service that includes the expertise of our ethical hackers, don't hesitate to contact us for our Continuous Hacking Squad Plan.