Friday, 19 July 2013

Nice business logic vulnerability and CSRF on the ASP.NET MVC Design Patterns book sample

Following a comment on this reddit threadt I did a search for 'ASP.NET MVC Design' patterns and found the site https://aspnetdesignpatterns.codeplex.com which is from the Wrox Professional ASP.NET Design Patterns book.

Since it looked like a nice MVC application, I grabbed a copy of the source code, upgraded it to .NET 4.0/MVC 4.0 (now on GitHub here) and had a quick look for MVC ModelBinding vulnerabilities.

And although it looks like the app is NOT vulnerable to MVC Model injections, that is mainly because there are very few controllers that use ModelBinding (i.e. that Design Pattern was not used (which ironically is my main recommendation to deal with MVC ModelBinding Vulnerabilities: don't use Model Binding :)  )).


Insecure Direct Object References

While looking for ModelBinding issues, I also had a look for the Top 10 2013-A4-Insecure Direct Object References and found a nice example of what not to do.

Here is the user page with the past orders

image

.... here is the page where I can see the 1st order (for the current user)

image

.... and here is the 2nd order:

image

... spot any probs so far?

Here is the controller that handles that request:

image

Since this controller asks for an orderId (which as seen above, in our current user’s case was 29 and 30), where is the code that checks that the requested orderId belongs to the current user?

image

.... well since in this case it is not there

image

... it is possible to see other customer’s orders (and shipping address)

image

Which makes for a nice example of an business logic vulnerability, where the request data is valid, but we are accessing information that shouldn't be visible to the current logged in user.

The solution for this vulnerability is to Use Indirect Object References

CSRF

This app is also vulnerable to CSRF (Cross Site Request Forgery)

Here is the raw HTTP Request (with only a valid session ID and Basket)

image

...which changed the current total of items we’re buying to 11:

image

Bad error handing:

This app also could do a much better job

image

... at handling malformed data

image

... with some errors only caught on the view (due to lack of data in the model)

image


XSS via product data

Talking about the views, this app does a pretty good job at using HtmlEncode on the views, except on this page:

image

Which means that XSS payloads could be introduced via the product database (normally managed via other systems/websites)

Is there more?

Probably, and it would be interesting to find and fix them (grab the code from GitHub and have a go)
Post a Comment