Rename adoptables (Security issue fixed!)

[kisazeky]

I included this

PHP Code:

$id = $_POST["id"];
$id = preg_replace("/[^a-zA-Z0-9s]/", "", $id);
$newname = $_POST["newname"];
$newname = preg_replace("/[^a-zA-Z0-9\\040.]/", "", $newname); 


What do I need to include to filter these injections, Rsmiley?

Edit: Maybe this will protect it?

Adding this to functions.php:

PHP Code:

function cleanQuery($string)
{
  if(get_magic_quotes_gpc())  // prevents duplicate backslashes
  {
    $string = stripslashes($string);
  }
  if (phpversion() >= '4.3.0')
  {
    $string = mysql_real_escape_string($string);
  }
  else
  {
    $string = mysql_escape_string($string);
  }
  return $string;
} 


Then adding this line in rename2.php:

PHP Code:

if (isset($_POST['newname'])) $newname = cleanQuery($_POST['newname']); 


What do you think Rsmiley?

It certainly does not hurt anything.