Sql injection in SMF [Admin section]

2006.09.08
Credit: Omid
Risk: Medium
Local: No
Remote: Yes
CWE: CWE-89


CVSS Base Score: 5.1/10
Impact Subscore: 6.4/10
Exploitability Subscore: 4.9/10
Exploit range: Remote
Attack complexity: High
Authentication: No required
Confidentiality impact: Partial
Integrity impact: Partial
Availability impact: Partial

Hi, There is a sql injection in SMF 1.1 RC3, in admin section : When an administrator is going to add a new board, the "cur_cat" parameter is not checked properly : File /Sources/ManageBoards.php, Line 609 : :: // Create a new board... :: if (isset($_POST['add'])) :: { :: // New boards by default go to the bottom of the category. :: if (empty($_POST['new_cat'])) <span class="quotelev2">>> $boardOptions['target_category'] = $_POST['cur_cat']; </span> :: if (!isset($boardOptions['move_to'])) :: $boardOptions['move_to'] = 'bottom'; :: <span class="quotelev2">>> createBoard($boardOptions); </span> :: } And in "createBoard()" function : File /Sources/Subs-Boards.php, Line 1095 : :: // Insert a board, the settings are dealt with later. :: db_query(" :: INSERT INTO {$db_prefix}boards :: (ID_CAT, name, description, boardOrder, memberGroups) <span class="quotelev2">>> VALUES ($boardOptions[target_category], SUBSTRING('$boardOptions[board_name]', 1, 255), '', 0, '-1,0')", __FILE__, __LINE__); </span> This is in administration section, so it doesnt seem to be critical. - Omid


Vote for this issue:
50%
50%


 

Thanks for you vote!


 

Thanks for you comment!
Your message is in quarantine 48 hours.

Comment it here.


(*) - required fields.  
{{ x.nick }} | Date: {{ x.ux * 1000 | date:'yyyy-MM-dd' }} {{ x.ux * 1000 | date:'HH:mm' }} CET+1
{{ x.comment }}

Copyright 2017, cxsecurity.com

 

Back to Top