All Downloads are FREE. Search and download functionalities are using the official Maven repository.

org.sonar.l10n.javascript.rules.javascript.S2245.html Maven / Gradle / Ivy

Go to download

There is a newer version: 10.17.0.28100
Show newest version
Using pseudorandom number generators (PRNGs) is security-sensitive. For example, it has led in the past to the following vulnerabilities:



When software generates predictable values in a context requiring unpredictability, it may be possible for an attacker to guess the next value that
will be generated, and use this guess to impersonate another user or access sensitive information.


As the Math.random() function relies on a weak pseudorandom number generator, this function should not be used for security-critical
applications or for protecting sensitive data. In such context, a cryptographically strong pseudorandom number generator (CSPRNG) should be used
instead.


Ask Yourself Whether


    
  
  •  the code using the generated value requires it to be unpredictable. It is the case for all encryption mechanisms or when a secret value, such
  as a password, is hashed. 
    • 
  
  •  the function you use generates a value which can be predicted (pseudo-random). 
    • 
  
  •  the generated value is used multiple times. 
    • 
  
  •  an attacker can access the generated value. 
    • 



There is a risk if you answered yes to any of those questions.


Recommended Secure Coding Practices


    
  
  •  Use a cryptographically strong pseudorandom number generator (CSPRNG) like crypto.getRandomValues(). 
    • 
  
  •  Use the generated random values only once. 
    • 
  
  •  You should not expose the generated random value. If you have to store it, make sure that the database or file is secure. 
    • 



Sensitive Code Example


const val = Math.random(); // Sensitive
// Check if val is used in a security context.



Compliant Solution


// === Client side ===
const crypto = window.crypto || window.msCrypto;
var array = new Uint32Array(1);
crypto.getRandomValues(array); // Compliant for security-sensitive use cases

// === Server side ===
const crypto = require('crypto');
const buf = crypto.randomBytes(1); // Compliant for security-sensitive use cases



See




© 2015 - 2024 Weber Informatics LLC | Privacy Policy