| Subject: | this class isn't safe for sql... |  
| Summary: | Package rating comment |  
| Messages: | 3 |  
| Author: | Andrea Venturi |  
| Date: | 2008-06-15 13:28:42 |  
| Update: | 2008-09-02 19:35:34 |  
|   |  
 
 | 
 | 
Andrea Venturi rated this package as follows:
| Utility:  | Good | 
| Consistency:  | Good | 
| Documentation:  | Good | 
| Examples:  | Good | 
| 
 | 
  Andrea Venturi - 2008-06-15 13:28:43  
this class isn't safe for sql injection, i added a small function for parsing inputs before use in production. 
  
  John Vaughan - 2008-08-22 16:46:25 -  In reply to message 1 from Andrea Venturi 
Hello Andrea, 
 
Would you mind posting your modifications?  I think this is a great class as well, but needs some SQL injection prevention to round it off.  You can also email me directly at jjvaughan at gmail 
 
Thanks for sharing! 
-John 
  
  Andrea Venturi - 2008-09-02 19:35:34 -  In reply to message 2 from John Vaughan 
I took the escape function from this class: http://www.phpclasses.org/browse/file/13783.html 
 
function sql_quote($value) 
{ 
    if( get_magic_quotes_gpc() ) 
    { 
      $value = stripslashes($value); 
    } 
     
    //check if this function exists 
    if( function_exists( 'mysql_real_escape_string' ) ) 
    { 
      $value = mysql_real_escape_string($value, $this->dbh); 
    } 
    //for PHP version < 4.3.0 use addslashes 
    else 
    { 
      $value = addslashes($value); 
    } 
    return $value; 
}  
  
   |