This site is the archived OWASP Foundation Wiki and is no longer accepting Account Requests.
To view the new OWASP Foundation website, please visit https://owasp.org

Difference between revisions of "Mass Assignment Cheat Sheet"

From OWASP
Jump to: navigation, search
(NodeJS update)
Line 66: Line 66:
 
* Attacker can guess common sensitive fields
 
* Attacker can guess common sensitive fields
 
* Attacker has access to source code and can review the models for sensitive fields
 
* Attacker has access to source code and can review the models for sensitive fields
* The object with sensitive fields has an empty constructor
+
* AND the object with sensitive fields has an empty constructor
 +
 
  
 
=== Case Studies ===
 
=== Case Studies ===
 
==== GitHub ====
 
==== GitHub ====
 
In 2012, GitHub was hacked using mass assignment. A user was able to upload his public key to any organization and thus make any subsequent changes in their repositories. [https://github.com/blog/1068-public-key-security-vulnerability-and-mitigation GitHub's Blog Post]
 
In 2012, GitHub was hacked using mass assignment. A user was able to upload his public key to any organization and thus make any subsequent changes in their repositories. [https://github.com/blog/1068-public-key-security-vulnerability-and-mitigation GitHub's Blog Post]
 +
  
 
=== Solutions ===
 
=== Solutions ===
Line 76: Line 78:
 
* Blacklist the non-bindable, sensitive fields
 
* Blacklist the non-bindable, sensitive fields
 
* Use Data Transfer Objects (DTOs)
 
* Use Data Transfer Objects (DTOs)
 +
  
 
= General Solutions =
 
= General Solutions =
Line 125: Line 128:
  
 
== NodeJS + Mongoose ==
 
== NodeJS + Mongoose ==
 +
 +
=== Whitelisting ===
 +
 +
  var UserSchema = new mongoose.Schema({
 +
    userid    : String,
 +
    password  : String,
 +
    email    : String,
 +
    admin    : Boolean,
 +
  });
 +
 
 +
  UserSchema.statics = {
 +
      User.userCreateSafeFields: ['userid', 'password', 'email']
 +
  };
 +
 
 +
  var User = mongoose.model('User', UserSchema);
 +
 +
  _ = require('underscore');
 +
  var user = new User(_.pick(req.body, User.userCreateSafeFields));
 +
 +
[http://underscorejs.org/#pick Reference]
 +
[https://nvisium.com/blog/2014/01/17/insecure-mass-assignment-prevention/ Reference]
 +
 +
=== Blacklisting ===
 +
 +
  var massAssign = require('mongoose-mass-assign');
 +
   
 +
  var UserSchema = new mongoose.Schema({
 +
    userid    : String,
 +
    password  : String,
 +
    email    : String,
 +
    admin    : { type: Boolean, protect: true, default: false }
 +
  });
 +
   
 +
  UserSchema.plugin(massAssign);
 +
   
 +
  var User = mongoose.model('User', UserSchema);
 +
 +
  /** Static method, useful for creation **/
 +
  var user = User.massAssign(req.body);
 +
 
 +
  /** Instance method, useful for updating  **/
 +
  var user = new User;
 +
  user.massAssign(req.body);
 +
 
 +
  /** Static massUpdate method **/
 +
  var input = { name: 'bhelx', admin: 'true' }; 
 +
  User.update({ '_id': someId }, { $set: User.massUpdate(input) }, console.log);
  
 
[https://www.npmjs.com/package/mongoose-mass-assign Reference]
 
[https://www.npmjs.com/package/mongoose-mass-assign Reference]

Revision as of 23:19, 10 March 2016

Cheatsheets-header.jpg

Last revision (mm/dd/yy): 03/10/2016

Introduction

Definition

Software frameworks sometime allow developers to automatically bind HTTP request parameters into program code variables or objects to make using that framework easier on developers. This can sometimes cause harm. Attackers can sometimes use this methodology to create new parameters that the developer never intended which in turn creates or overwrites new variable or objects in program code that was not intended. This is called a mass assignment vulnerability.

Alternative Names

Depending on the language/framework in question, this vulnerability can have several alternative names

  • Mass Assignment: Ruby on Rails, NodeJS
  • Autobinding: Spring MVC, ASP.NET MVC
  • Object injection: PHP

Example

Suppose there is a form for editing a user's account information:

  <form>
     <input name=userid type=text>
     <input name=password type=text>
     <input name=email text=text>
     <input type=submit>
  </form>

Here is the object that the form is binding to:

  public class User {
     private String userid;
     private String password;
     private String email;
     private boolean isAdmin;
   
     //Getters & Setters
   }

Here is the controller handling the request:

  @RequestMapping(value = "/addUser, method = RequestMethod.POST)
  public String submit(User user) {
     
     userService.add(user);
  
     return "successPage";
  }

Here is the typical request:

  POST /addUser
  
  userid=bobbytables&password=hashedpass&[email protected]

And here is the exploit:

  POST /addUser
  
  userid=bobbytables&password=hashedpass&[email protected]&isAdmin=true

Exploitability

This functionality becomes exploitable when:

  • Attacker can guess common sensitive fields
  • Attacker has access to source code and can review the models for sensitive fields
  • AND the object with sensitive fields has an empty constructor


Case Studies

GitHub

In 2012, GitHub was hacked using mass assignment. A user was able to upload his public key to any organization and thus make any subsequent changes in their repositories. GitHub's Blog Post


Solutions

  • Whitelist the bindable, non-sensitive fields
  • Blacklist the non-bindable, sensitive fields
  • Use Data Transfer Objects (DTOs)


General Solutions

Data Transfer Objects (DTOs)

An architectural approach is to create Data Transfer Objects and avoid binding input directly to domain objects. Only the fields that are meant to be editable by the user are included in the DTO.

  public class UserRegistrationFormDTO {
     private String userid;
     private String password;
     private String email;
  
     //private boolean isAdmin;
   
     //Getters & Setters
   }

Language & Framework Specific Solutions

Spring MVC

Whitelisting

  @Controller
  public class UserController
  {
     @InitBinder
     public void initBinder(WebDataBinder binder, WebRequest request)
     {
        binder.setAllowedFields(["userid","password","email"]);
     }
  
     ...
  }

Reference

Blacklisting

  @Controller
  public class UserController
  {
     @InitBinder
     public void initBinder(WebDataBinder binder, WebRequest request)
     {
        binder.setDisallowedFields(["isAdmin"]);
     }
  
     ...
  }

Reference

NodeJS + Mongoose

Whitelisting

  var UserSchema = new mongoose.Schema({
    userid    : String,
    password  : String,
    email     : String,
    admin     : Boolean,
  });
  
  UserSchema.statics = {
      User.userCreateSafeFields: ['userid', 'password', 'email']
  };
  
  var User = mongoose.model('User', UserSchema);
  _ = require('underscore');
  var user = new User(_.pick(req.body, User.userCreateSafeFields));

Reference Reference

Blacklisting

  var massAssign = require('mongoose-mass-assign');
   
  var UserSchema = new mongoose.Schema({
    userid    : String,
    password  : String,
    email     : String,
    admin     : { type: Boolean, protect: true, default: false }
  });
   
  UserSchema.plugin(massAssign);
   
  var User = mongoose.model('User', UserSchema);
  /** Static method, useful for creation **/
  var user = User.massAssign(req.body);
  
  /** Instance method, useful for updating  **/
  var user = new User;
  user.massAssign(req.body);
  
  /** Static massUpdate method **/
  var input = { name: 'bhelx', admin: 'true' };  
  User.update({ '_id': someId }, { $set: User.massUpdate(input) }, console.log);

Reference

Ruby On Rails

Reference

Django

Reference

ASP.NET

Reference

PHP Laravel + Eloquent

Whitelisting

  <?php
  
  namespace App;
  
  use Illuminate\Database\Eloquent\Model;
  
  class User extends Model
  {
     private $userid;
     private $password;
     private $email;
     private $isAdmin;
  
     protected $fillable = array('userid','password','email');
  
  }

Reference

Blacklisting

  <?php
  
  namespace App;
  
  use Illuminate\Database\Eloquent\Model;
  
  class User extends Model
  {
     private $userid;
     private $password;
     private $email;
     private $isAdmin;
  
     protected $guarded = array('isAdmin');
  
  }

Reference

Grails

Reference

Play

Reference

Jackson (JSON Object Mapper)

Reference Reference

GSON (JSON Object Mapper)

Reference Reference

JSON-Lib (JSON Object Mapper)

Reference

Flexjson (JSON Object Mapper)

Reference

Authors and Primary Editors

References and future reading

Other Cheatsheets