Commit 7f3faffc authored by James Hawkins's avatar James Hawkins Committed by Alexandre Julliard

msi: Don't allow nested quotes in SQL queries.

parent e821f9d7
...@@ -5257,10 +5257,7 @@ static void test_quotes(void) ...@@ -5257,10 +5257,7 @@ static void test_quotes(void)
query = "INSERT INTO `Table` ( `A` ) VALUES ( 'This is a ''string'' ok' )"; query = "INSERT INTO `Table` ( `A` ) VALUES ( 'This is a ''string'' ok' )";
r = run_query(hdb, 0, query); r = run_query(hdb, 0, query);
todo_wine ok(r == ERROR_BAD_QUERY_SYNTAX, "Expected ERROR_BAD_QUERY_SYNTAX, got %d\n", r);
{
ok(r == ERROR_BAD_QUERY_SYNTAX, "Expected ERROR_BAD_QUERY_SYNTAX, got %d\n", r);
}
query = "INSERT INTO `Table` ( `A` ) VALUES ( 'This is a '''string''' ok' )"; query = "INSERT INTO `Table` ( `A` ) VALUES ( 'This is a '''string''' ok' )";
r = run_query(hdb, 0, query); r = run_query(hdb, 0, query);
...@@ -5287,19 +5284,13 @@ static void test_quotes(void) ...@@ -5287,19 +5284,13 @@ static void test_quotes(void)
size = MAX_PATH; size = MAX_PATH;
r = MsiRecordGetString(hrec, 1, buf, &size); r = MsiRecordGetString(hrec, 1, buf, &size);
ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r); ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r);
todo_wine ok(!lstrcmp(buf, "This is a \"string\" ok"),
{ "Expected \"This is a \"string\" ok\", got %s\n", buf);
ok(!lstrcmp(buf, "This is a \"string\" ok"),
"Expected \"This is a \"string\" ok\", got %s\n", buf);
}
MsiCloseHandle(hrec); MsiCloseHandle(hrec);
r = MsiViewFetch(hview, &hrec); r = MsiViewFetch(hview, &hrec);
todo_wine ok(r == ERROR_NO_MORE_ITEMS, "Expected ERROR_NO_MORE_ITEMS, got %d\n", r);
{
ok(r == ERROR_NO_MORE_ITEMS, "Expected ERROR_NO_MORE_ITEMS, got %d\n", r);
}
MsiViewClose(hview); MsiViewClose(hview);
MsiCloseHandle(hview); MsiCloseHandle(hview);
......
...@@ -254,13 +254,8 @@ int sqliteGetToken(const WCHAR *z, int *tokenType){ ...@@ -254,13 +254,8 @@ int sqliteGetToken(const WCHAR *z, int *tokenType){
case '`': case '\'': { case '`': case '\'': {
int delim = z[0]; int delim = z[0];
for(i=1; z[i]; i++){ for(i=1; z[i]; i++){
if( z[i]==delim ){ if( z[i]==delim )
if( z[i+1]==delim ){ break;
i++;
}else{
break;
}
}
} }
if( z[i] ) i++; if( z[i] ) i++;
if( delim == '`' ) if( delim == '`' )
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment