msi: Reject greater than and less than string comparisons in WHERE clauses.
Hans Leidekker
hans at codeweavers.com
Tue Dec 15 02:50:40 CST 2009
See http://bugs.winehq.org/show_bug.cgi?id=19337
---
dlls/msi/tests/db.c | 24 ++++++++++++++++++++++++
dlls/msi/where.c | 6 +-----
2 files changed, 25 insertions(+), 5 deletions(-)
diff --git a/dlls/msi/tests/db.c b/dlls/msi/tests/db.c
index bafa0db..9837866 100644
--- a/dlls/msi/tests/db.c
+++ b/dlls/msi/tests/db.c
@@ -1761,6 +1761,30 @@ static void test_where(void)
ok( r == ERROR_SUCCESS, "query failed: %d\n", r );
MsiCloseHandle( rec );
+ rec = 0;
+ query = "SELECT * FROM `Media` WHERE `DiskPrompt` < 'Cabinet'";
+ r = do_query(hdb, query, &rec);
+ ok( r == ERROR_BAD_QUERY_SYNTAX, "query failed: %d\n", r );
+ MsiCloseHandle( rec );
+
+ rec = 0;
+ query = "SELECT * FROM `Media` WHERE `DiskPrompt` > 'Cabinet'";
+ r = do_query(hdb, query, &rec);
+ ok( r == ERROR_BAD_QUERY_SYNTAX, "query failed: %d\n", r );
+ MsiCloseHandle( rec );
+
+ rec = 0;
+ query = "SELECT * FROM `Media` WHERE `DiskPrompt` <> 'Cabinet'";
+ r = do_query(hdb, query, &rec);
+ todo_wine ok( r == ERROR_SUCCESS, "query failed: %d\n", r );
+ MsiCloseHandle( rec );
+
+ rec = 0;
+ query = "SELECT * FROM `Media` WHERE `DiskPrompt` = 'Cabinet'";
+ r = do_query(hdb, query, &rec);
+ ok( r == ERROR_NO_MORE_ITEMS, "query failed: %d\n", r );
+ MsiCloseHandle( rec );
+
rec = MsiCreateRecord(1);
MsiRecordSetString(rec, 1, "");
diff --git a/dlls/msi/where.c b/dlls/msi/where.c
index 32f71b9..545d6a8 100644
--- a/dlls/msi/where.c
+++ b/dlls/msi/where.c
@@ -309,9 +309,7 @@ static UINT STRCMP_Evaluate( MSIWHEREVIEW *wv, UINT row, const struct expr *cond
sr = lstrcmpW( l_str, r_str );
*val = ( cond->u.expr.op == OP_EQ && ( sr == 0 ) ) ||
- ( cond->u.expr.op == OP_NE && ( sr != 0 ) ) ||
- ( cond->u.expr.op == OP_LT && ( sr < 0 ) ) ||
- ( cond->u.expr.op == OP_GT && ( sr > 0 ) );
+ ( cond->u.expr.op == OP_NE && ( sr != 0 ) );
return ERROR_SUCCESS;
}
@@ -644,8 +642,6 @@ static UINT WHERE_VerifyCondition( MSIDATABASE *db, MSIVIEW *table, struct expr
switch( cond->u.expr.op )
{
case OP_EQ:
- case OP_GT:
- case OP_LT:
case OP_NE:
break;
default:
--
1.6.3.3
More information about the wine-patches
mailing list