Bug 33612

Summary:

Use the CaseFoldingHash function in DatabaseAuthorizer to store the list of whitelisted sqlite functions

Product:

WebKit

Reporter:

Dumitru Daniliuc <dumi>

Component:

New Bugs

Assignee:

Dumitru Daniliuc <dumi>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, darin

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

OS X 10.5

Attachments:

Description	Flags
patch	darin: review+, dumi: commit-queue-
patch	darin: review+, dumi: commit-queue-

Description Dumitru Daniliuc 2010-01-13 12:01:54 PST

DatabaseAuthorizer should use a CaseFoldingHash function to store the list of whitelisted sqlite functions, instead of doing m_whitelistedFunctions.contains(functionName.lower()). Also, the indenting in DatabaseAuthorizer::allowFunction() should be fixed.

Comment 1 Dumitru Daniliuc 2010-01-13 12:15:21 PST

Created attachment 46483 [details]
patch

Comment 2 Darin Adler 2010-01-13 12:23:06 PST

Comment on attachment 46483 [details]
patch

Code change is fine. But normally we include the change log entry in the patch for review.

Comment 3 Dumitru Daniliuc 2010-01-13 12:27:28 PST

Created attachment 46486 [details]
patch

(In reply to comment #2)
> (From update of attachment 46483 [details])
> Code change is fine. But normally we include the change log entry in the patch
> for review.

D'oh! I knew something was missing from this patch... Please take a look at the new one.

Comment 4 Dumitru Daniliuc 2010-01-13 13:09:46 PST

Landed as r53198.