------------------------------------------------------------------------ Ultimate Member Local File Inclusion vulnerability ------------------------------------------------------------------------ Burak Kelebek, July 2016 ------------------------------------------------------------------------ Abstract ------------------------------------------------------------------------ It was discovered that Ultimate Member [2] is vulnerable to PHP File Inclusion [3]. In order to exploit this issue an attacker must be able to place an arbitrary PHP file on the target system. Afterwards the attacker needs to lure an authenticated admin to visit a malicious page. Through CSRF the attacker could compromise WordPress, by executing the malicious PHP file. ------------------------------------------------------------------------ OVE ID ------------------------------------------------------------------------ OVE-20160712-0011 ------------------------------------------------------------------------ Tested versions ------------------------------------------------------------------------ This issue was succesfully tested on the Ultimate Member [2] WordPress plugin version 1.3.64. ------------------------------------------------------------------------ Fix ------------------------------------------------------------------------ There is a fix available in Pre-released version: pre-v1.3.65.10 [4]. After the fix is applied user input is no more used when including local files in WordPress. ------------------------------------------------------------------------ Introduction ------------------------------------------------------------------------ The File Inclusion vulnerability allows an attacker to include a file, exploiting a "dynamic file inclusion" mechanisms implemented in the Ultimate Member plugin of WordPress. ------------------------------------------------------------------------ Details ------------------------------------------------------------------------ The vulnerability occurs due to the use of user-supplied input in the 'page' parameter without proper validation. The code first checks if 'ultimatemember-' exists in page parameter supplied by user. If it exists it deletes 'ultimatemember-' and assigns user input to the template parameter. Lastly it inserts user input in the PHP function include_once [5]. ultimate-member/admin/core/um-admin-dashboard.php: $page = $_REQUEST['page']; [..] else if ( strstr( $page, 'ultimatemember-' ) ) { $template = str_replace('ultimatemember-','',$page); $file = um_path . 'admin/templates/welcome/'. $template . '.php'; if ( file_exists( $file ) ){ include_once um_path . 'admin/templates/welcome/'. $template . '.php'; } } ------------------------------------------------------------------------ Proof of concept ------------------------------------------------------------------------ Proof of Concept below simply opens a file in the temp folder of WordPress. ------------------------------------------------------------------------ References ------------------------------------------------------------------------ [1] https://sumofpwn.nl/advisory/2016/ultimate_member_local_file_inclusion_vulnerability.html [2] https://wordpress.org/plugins/ultimate-member/ [3] https://www.owasp.org/index.php/PHP_File_Inclusion [4] https://github.com/ultimatemember/ultimatemember/commit/50f5828ac7fa08a1afa0a52b456a1db49071c804 [5] http://php.net/manual/en/function.include-once.php