Forum Moderators: coopster

Message Too Old, No Replies

sql injection

         

ayushchd

4:23 pm on Sep 6, 2007 (gmt 0)

10+ Year Member



Is it enough to use mysql_escape_string to safeguard from sql injection attacks?

ayushchd

5:09 pm on Sep 8, 2007 (gmt 0)

10+ Year Member



Thanks.

Can you tell me what's wrong with the function I posted in my previous reply?

henry0

1:39 pm on Sep 9, 2007 (gmt 0)

WebmasterWorld Senior Member 10+ Year Member



What seems to be the problem

run my test it works, escape ok and does not return any error


<?

function db_connect()
{
$result = @mysql_pconnect("aaaa", "bbbb", "sssss");
if (!isset($result) && empty($result))
{echo "can't connect!"; }
if (!@mysql_select_db("#*$!xx"))
return false;
return $result;
}

$conn=db_connect();

function SqlClean($my_var)
{
if (get_magic_quotes_gpc())
{
if (is_array($my_var)) {
foreach ($my_var as $key) {
stripslashes($key);
}
} else {
stripslashes($my_var);
}
}
if (is_array($my_var)) {
foreach ($my_var as $key) {
$key = mysql_real_escape_string($key);
return $key;
}
} else {
$my_var = mysql_real_escape_string($my_var);
}
return $my_var;
}

$a=" 'asass' ";
$a=SqlClean($a);
echo" test: $a";
?>

This 32 message thread spans 2 pages: 32