=======================
Exercise: Apache Struts
=======================

.. container:: subheading

   Unsafe deserialization leading to an RCE

   CVE-2017-9805

.. rst-class:: setup

Setup
=====

For this example you should download:

- `CodeQL for Visual Studio Code <https://help.semmle.com/codeql/codeql-for-vscode/procedures/setting-up.html>`__
- `Apache Struts database <https://downloads.lgtm.com/snapshots/java/apache/struts/apache-struts-7fd1622-CVE-2018-11776.zip>`__

.. note::

   For this example, we will be analyzing `Apache Struts <https://github.com/apache/struts>`__.

   You can also query the project in `the query console <https://lgtm.com/query/project:1878521151/lang:java/>`__ on LGTM.com.

   .. insert database-note.rst to explain differences between database available to download and the version available in the query console.

   .. include:: ../slide-snippets/database-note.rst

   .. resume slides

Unsafe deserialization in Struts
================================

Apache Struts provides a ``ContentTypeHandler`` interface, which can be implemented for specific content types. It defines the following interface method:

.. code-block:: java

  void toObject(Reader in, Object target);


which is intended to populate the ``target`` object with data from the reader, usually through deserialization. However, the ``in`` parameter should be considered untrusted, and should not be deserialized without sanitization.

RCE in Apache Struts
====================

- Vulnerable code looked like this (`original <https://lgtm.com/projects/g/apache/struts/snapshot/b434c23f95e0f9d5bde789bfa07f8fc1d5a8951d/files/plugins/rest/src/main/java/org/apache/struts2/rest/handler/XStreamHandler.java?sort=name&dir=ASC&mode=heatmap#L45>`__):

   .. code-block:: java
   
      public void toObject(Reader in, Object target) {
        XStream xstream = createXStream();
        xstream.fromXML(in, target);
      }

- Xstream allows deserialization of **dynamic proxies**, which permit remote code execution.

- Disclosed as `CVE-2017-9805 <http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2017-9805>`__

- Blog post: https://securitylab.github.com/research/apache-struts-vulnerability-cve-2017-9805

Finding the RCE yourself
========================

#. Create a class to find the interface ``org.apache.struts2.rest.handler.ContentTypeHandler``

   **Hint**: Use predicate ``hasQualifiedName(...)``

#. Identify methods called ``toObject``, which are defined on direct subtypes of ``ContentTypeHandler``

   **Hint**: Use ``Method.getDeclaringType()`` and ``Type.getASupertype()``

#. Implement a ``DataFlow::Configuration``, defining the source as the first parameter of a ``toObject`` method, and the sink as an instance of ``UnsafeDeserializationSink``.

   **Hint**: Use ``Node::asParameter()``

#. Construct the query as a path-problem query, and verify you find one result.

Model answer, step 1
====================

.. code-block:: ql

  import java

  /** The interface `org.apache.struts2.rest.handler.ContentTypeHandler`. */

  class ContentTypeHandler extends RefType {
    ContentTypeHandler() {
      this.hasQualifiedName("org.apache.struts2.rest.handler", "ContentTypeHandler")
    }
  }

Model answer, step 2
====================

.. code-block:: ql

   /** A `toObject` method on a subtype of `org.apache.struts2.rest.handler.ContentTypeHandler`. */
   class ContentTypeHandlerDeserialization extends Method {
     ContentTypeHandlerDeserialization() {
       this.getDeclaringType().getASupertype() instanceof ContentTypeHandler and
       this.hasName("toObject")

Model answer, step 3
====================

.. code-block:: ql

   import UnsafeDeserialization
   import semmle.code.java.dataflow.DataFlow::DataFlow
   /**
    * Configuration that tracks the flow of taint from the first parameter of
    * `ContentTypeHandler.toObject` to an instance of unsafe deserialization.
    */
   class StrutsUnsafeDeserializationConfig extends Configuration {
     StrutsUnsafeDeserializationConfig() { this = "StrutsUnsafeDeserializationConfig" }
     override predicate isSource(Node source) {
       source.asParameter() = any(ContentTypeHandlerDeserialization des).getParameter(0)
     }
     override predicate isSink(Node sink) { sink instanceof UnsafeDeserializationSink }
   }

Model answer, step 4
====================

.. code-block:: ql

   import PathGraph
   ...
   from PathNode source, PathNode sink, StrutsUnsafeDeserializationConfig conf
   where conf.hasFlowPath(source, sink)
     and sink.getNode() instanceof UnsafeDeserializationSink
   select sink.getNode().(UnsafeDeserializationSink).getMethodAccess(), source, sink, "Unsafe    deserialization of $@.", source, "user input"

More full-featured version: https://github.com/github/security-lab/tree/master/CodeQL_Queries/java/Apache_Struts_CVE-2017-9805