Download & Extend

db_escape_string() function is not necessary

Project:Delicious
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:active
Issue tags:escape, query

Issue Summary

db_escape_string() function is not necessary in delicious.inc @ line 136 (and more) because you use %-modifiers with arguments in SQL queries. In your case, you double escape simple quotes.

User-supplied arguments to the query should be passed in as separate parameters so that they can be properly escaped to avoid SQL injection attacks.
Valid %-modifiers are: %s, %d, %f, %b (binary data, do not enclose in '') and %%.

http://api.drupal.org/api/function/db_query/6

nobody click here