HOR-4751
This commit is contained in:
committed by
Julio Cesar Laura Avendaño
parent
adcbbbed97
commit
3d9380dbd6
@@ -352,6 +352,7 @@ class InputFilter
|
||||
* @param String $source
|
||||
* @param Resource $connection - An open MySQL connection
|
||||
* @return String $source
|
||||
* @todo We need to review this method, because the sended string is unescaped
|
||||
*/
|
||||
public function escapeString($string, &$connection)
|
||||
{
|
||||
@@ -365,6 +366,19 @@ class InputFilter
|
||||
return $string;
|
||||
}
|
||||
|
||||
/*
|
||||
* Escapes a string using a Propel connection
|
||||
*
|
||||
* @param string $string The string to escapes
|
||||
* @param object $connection The connection object
|
||||
*
|
||||
* @return string
|
||||
*/
|
||||
public function escapeUsingConnection($string, $connection)
|
||||
{
|
||||
return mysql_real_escape_string($string, $connection->getResource());
|
||||
}
|
||||
|
||||
/**
|
||||
* Internal method removes tags/special characters
|
||||
* @author Marcelo Cuiza
|
||||
|
||||
@@ -44,6 +44,23 @@ class Applications
|
||||
|
||||
//Start the connection to database
|
||||
$con = Propel::getConnection(AppDelegationPeer::DATABASE_NAME);
|
||||
|
||||
//Sanitize input variables
|
||||
$inputFilter = new InputFilter();
|
||||
$userUid = $inputFilter->validateInput($userUid, 'int');
|
||||
$start = $inputFilter->validateInput($start, 'int');
|
||||
$limit = $inputFilter->validateInput($limit, 'int');
|
||||
$search = $inputFilter->escapeUsingConnection($search, $con);
|
||||
$process = $inputFilter->validateInput($process, 'int');
|
||||
//$status doesn't require sanitization
|
||||
$dir = in_array($dir, ['ASC', 'DESC']) ? $dir :'DESC';
|
||||
$sort = $inputFilter->escapeUsingConnection($sort, $con);
|
||||
$category = $inputFilter->escapeUsingConnection($category, $con);
|
||||
$dateFrom = $inputFilter->escapeUsingConnection($dateFrom, $con);
|
||||
$dateTo = $inputFilter->escapeUsingConnection($dateTo, $con);
|
||||
$columnSearch = $inputFilter->escapeUsingConnection($columnSearch, $con);
|
||||
|
||||
//Start the transaction
|
||||
$con->begin();
|
||||
$stmt = $con->createStatement();
|
||||
|
||||
|
||||
Reference in New Issue
Block a user