[AppDB] - protect against sql injection in select, update and delete statements

Tony Lambregts tony.lambregts at gmail.com
Sun Jun 25 16:37:23 CDT 2006


Chris Morgan wrote:
> Protect against sql injection attacks in select, update and delete statements 
> by using query_parameters().  mysql_real_escape_string() is used on variables 
> in cases where using query_parameters() isn't possible due to the complexity 
> of the query. These could potentially be simplified so query_parameters() 
> could be used.
>
>   
This patch is quite large  and it breaks moving an test results to an 
existing version.

I had spent a fair amount of time testing this before I found this bug 
and I think that most of it may be OK but  as they say "one drop of crap 
in a gallon of wine..."

Can you  submit some smaller chunks of this so that I can test them and 
we get them in.  We already have query_parameters()  in the code base so 
the simple stuff like addcomment.php is probably good to go. If you 
could submit a patch for just those I would appreciate it.

--

Tony Lambregts






More information about the wine-patches mailing list