ally-petitt/cve-2023-45503CVE-2023-45503 Referenz
In Macrob7 Macs Framework Content Management System (CMS) Version 1.1.4f und früher führt unsichere Handhabung von Benutzereingaben zu 16 SQL-Injection-Schwachstellen. Die Möglichkeit, beliebige SQL-Abfragen auszuführen, kann zur Offenlegung privater Daten führen, einschließlich der Passwort-Hashes der Benutzer und der Möglichkeit, die Anmeldeinformationen und Privilegien anderer Benutzer zu ändern. Die Auswirkungen können eine Privilegieneskalation und potenzielle Remote-Code-Ausführung (RCE) umfassen.
Eine Tabelle, die jeden betroffenen Endpunkt, den anfälligen Parameter und die anfälligen Funktionen auflistet, finden Sie unter diesem Link.
CWE-Klassifizierung: CWE-89: Fehlende Neutralisierung von speziellen Elementen in einem SQL-Befehl ('SQL-Injection')
Gemeldet von: Ally Petitt
Betroffenes Produkt: Macrob7 Macs CMS
Betroffene Versionen: 1.1.4f und früher
In 16 Fällen, die ich entdeckte, wurden Benutzereingaben ohne ordnungsgemäße Bereinigung oder Parametrisierung entgegengenommen. Beispielsweise wird in der Funktion „Passwort vergessen" dieses CMS eine E-Mail-Adresse angefordert.
Application/plugins/CMS/controllers/CMS.php:224
public function forgotPasswordProcess()
{
$this->loadModels();
$emailAddress = Post::getByKey('emailAddress');
$user = $this->usersModel->getUserByEmailAddress($emailAddress);
Die E-Mail-Adresse wird dann an die Methode getUserByEmailAddress() übergeben, die die E-Mail an weitergibt, eine Methode, die nicht ordnungsgemäß gegen injizierte SQL-Abfragen schützt.
selectSingle()Application/plugins/CMS/models/Users_Model.php:41
public function getUserByEmailAddress($emailAddress)
{
return $this->selectSingle( $this->getCMSTableNameUsers(), array('EmailAddress'=>$emailAddress));
}
Dann ruft selectSingle() die Funktion select() mit der übergebenen Benutzereingabe auf.
Application/core/DB.php:200
public function selectSingle($tableName, array $where = array(), array $fields = array('*'))
{
$return = $this->select($tableName, $where, $fields);
$single = NULL;
if( count($return) > 0 )
$single = $return[0];
return $single;
}
Die anfällige Funktion select() verkettet die übergebenen Parameter zu einer SQL-Abfrage, die anschließend ausgeführt wird.
Application/core/DB.php:186
public function select($tableName, array $where = array(), array $fields = array('*'))
{
$fieldsString = $this->generatePair($fields, ',');
$whereString = $this->generateKeyValuePair($where, '=', 'AND');
$sql = 'SELECT '.$fieldsString.' FROM '.$tableName;
if($whereString !='')
$whereString = ' WHERE '.$whereString;
$sql = $sql.' '.$whereString.';';
return $this->execute($sql)->fetchAll($this->returnType, $this->className);
}
Aufgrund des Fehlens von Eingabebereinigung, Validierung und Parametrisierung bleibt diese Funktion anfällig für SQL-Injection-Angriffe.
Leider bedeutet die mangelnde Wartung dieses CMS, dass kein gepatchtes Update verfügbar ist. Einzelne Benutzer können den Code ändern, um parametrisierte SQL-Abfragen zu verwenden, anstatt sich auf Verkettung zu verlassen, um Benutzereingaben an die Datenbank zu übergeben. Prepared Statements sind ein Beispiel für eine Gegenmaßnahme, die dies erreichen kann.