[Bug 11479] New: db: \r should be stripped from identifiers in SQL

http://bugs.winehq.org/show_bug.cgi?id=11479

           Summary: db: \r should be stripped from identifiers in SQL
           Product: Wine
           Version: CVS/GIT
          Platform: Other
        OS/Version: other
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: P2
         Component: msi
        AssignedTo: wine-bugs(a)winehq.org
        ReportedBy: jdahlin(a)async.com.br


Created an attachment (id=10620)
 --> (http://bugs.winehq.org/attachment.cgi?id=10620)
stupid patch and testcase

While trying to compile the basic example in Wix, an query string is sent in to
MsiDatabaseOpenView() for which MSI_ParseSQL return 1. Eg:

trace:msi:MSI_DatabaseOpenViewW L"CREATE TABLE `_SummaryInformation` (
`PropertyId` INT NOT NULL, `Value\r` CHAR(255\r) NOT NULL LOCALIZABLE PRIMARY
KEY `PropertyId\r`)" 0x61ed98
trace:msi:MSI_ParseSQL Parse returned 1

To reproduce this error, the instructions in bug 11414, comment 10 applies.
Attached is a new test to msi/tests/db.c stupid and naive patch which just
strips the trailing \r if present.


-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.