Headline
CVE-2016-15020: Fixed orderBy injection vulnerability · liftkit/database@42ec8f2
A vulnerability was found in liftkit database up to 2.13.1. It has been classified as critical. This affects the function processOrderBy of the file src/Query/Query.php. The manipulation leads to sql injection. Upgrading to version 2.13.2 is able to address this issue. The name of the patch is 42ec8f2b22e0b0b98fb5b4444ed451c1b21d125a. It is recommended to upgrade the affected component. The associated identifier of this vulnerability is VDB-218391.
@@ -921,7 +921,7 @@ public function groupBy ($field)
}
public function orderBy ($field, $direction = ‘ASC’)
public function orderBy ($field, $direction = self::QUERY_ORDER_ASC)
{
$this->orderBys[] = array(
‘field’ => $this->filterIdentifier($field),
@@ -1064,7 +1064,11 @@ protected function processOrderBy ()
$orderBys = array();
foreach ($this->orderBys as $orderBy) {
$orderBys[] = $orderBy[‘field’] . ' ' . $orderBy[‘direction’];
$direction = (strtoupper($orderBy[‘direction’]) == self::QUERY_ORDER_DESC)
? self::QUERY_ORDER_DESC
: self::QUERY_ORDER_ASC;
$orderBys[] = $orderBy[‘field’] . ' ' . $direction;
}
return $sql.implode(', ', $orderBys);